This is an automated email from the ASF dual-hosted git repository.
jamesbognar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/juneau.git
The following commit(s) were added to refs/heads/master by this push:
new f6d03dd302 TODO-238 — BUG-15: add null-guards to
RestClient.request(method,url) (documented-non-null url was dereferenced
unguarded, backing all verb methods + remote()).
f6d03dd302 is described below
commit f6d03dd302e9c8344bd6833cb2a0be0ba96fe63e
Author: James Bognar <[email protected]>
AuthorDate: Fri Jul 17 16:08:34 2026 -0400
TODO-238 — BUG-15: add null-guards to RestClient.request(method,url)
(documented-non-null url was dereferenced unguarded, backing all verb methods +
remote()).
Co-authored-by: Cursor <[email protected]>
---
.../src/main/java/org/apache/juneau/rest/client/RestClient.java | 6 ++++++
1 file changed, 6 insertions(+)
diff --git
a/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
b/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
index 49c30edc0e..064fe72b62 100644
---
a/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
+++
b/juneau-rest/juneau-rest-client/src/main/java/org/apache/juneau/rest/client/RestClient.java
@@ -84,6 +84,10 @@ public final class RestClient implements Closeable {
BodyConverter.of(File.class, file ->
TransportBody.of(FileBody.of(file)))
);
+ // Argument name constants for assertArgNotNull
+ private static final String ARG_method = "method";
+ private static final String ARG_url = "url";
+
final HttpTransport transport;
final List<HttpHeader> defaultHeaders;
final List<HttpPart> defaultQueryData;
@@ -204,6 +208,8 @@ public final class RestClient implements Closeable {
* @return A new {@link RestRequest}. Never <jk>null</jk>.
*/
public RestRequest request(String method, String url) {
+ assertArgNotNull(ARG_method, method);
+ assertArgNotNull(ARG_url, url);
var resolvedUrl = rootUrl != null && !url.contains("://") ?
rootUrl + url : url;
return new RestRequest(this, method, resolvedUrl);
}