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);
        }

Reply via email to