You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by "coheigea (via GitHub)" <gi...@apache.org> on 2026/07/07 10:24:25 UTC

[PR] Plug URIResolver into AttachmentUtil [cxf]

coheigea opened a new pull request, #3289:
URL: https://github.com/apache/cxf/pull/3289

   (no comment)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Plug URIResolver into AttachmentUtil [cxf]

Posted by "github-advanced-security[bot] (via GitHub)" <gi...@apache.org>.
github-advanced-security[bot] commented on code in PR #3289:
URL: https://github.com/apache/cxf/pull/3289#discussion_r3535733421


##########
core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java:
##########
@@ -593,12 +594,16 @@
                     final boolean followUrls = Boolean.valueOf(SystemPropertyAction
                         .getProperty(ATTACHMENT_XOP_FOLLOW_URLS_PROPERTY, "false"));
                     if (followUrls) {
-                        return new URLDataSource(new URL(contentId));
+                        final URL remoteUrl = new URL(contentId);
+                        URIResolver.checkAllowedScheme(remoteUrl);
+                        return new URLDataSource(remoteUrl);

Review Comment:
   ## CodeQL / Server-side request forgery
   
   Potential server-side request forgery due to a [user-provided value](1).
   Potential server-side request forgery due to a [user-provided value](2).
   Potential server-side request forgery due to a [user-provided value](3).
   Potential server-side request forgery due to a [user-provided value](4).
   
   [Show more details](https://github.com/apache/cxf/security/code-scanning/2242)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Plug URIResolver into AttachmentUtil [cxf]

Posted by "coheigea (via GitHub)" <gi...@apache.org>.
coheigea merged PR #3289:
URL: https://github.com/apache/cxf/pull/3289


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org