Merge pull request data-integrations#195 from cloudsufi/null-pointer-fix

vikasrathee-cs · web-flow · commit 769b221bb650 · 2025-06-02T10:24:03.000+05:30
[PLUGIN-1894] Add handling for Null pointer exception in case of Null response
diff --git a/src/main/java/io/cdap/plugin/http/common/pagination/BaseHttpPaginationIterator.java b/src/main/java/io/cdap/plugin/http/common/pagination/BaseHttpPaginationIterator.java
@@ -165,7 +165,7 @@ protected BasePage getNextPage() {
             config.getPaginationType()));
         }
         LOG.warn(String.format("Fetching from url '%s' returned status code '%d' and body '%s'",
-                               nextPageUrl, httpStatusCode, response.getBody()));
+                               nextPageUrl, httpStatusCode, response == null ? null : response.getBody()));
         // this will be handled by PageFactory. Here no handling is needed.
         break;
       default:

Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,7 @@ protected BasePage getNextPage() {`
`165`	`165`	`config.getPaginationType()));`
`166`	`166`	`}`
`167`	`167`	`LOG.warn(String.format("Fetching from url '%s' returned status code '%d' and body '%s'",`
`168`		`- nextPageUrl, httpStatusCode, response.getBody()));`
	`168`	`+ nextPageUrl, httpStatusCode, response == null ? null : response.getBody()));`
`169`	`169`	`// this will be handled by PageFactory. Here no handling is needed.`
`170`	`170`	`break;`
`171`	`171`	`default:`