gnodet-bot commented on code in PR #26962:
URL: https://github.com/apache/camel/pull/26962#discussion_r4119631400


##########
core/camel-util/src/main/java/org/apache/camel/util/URISupport.java:
##########
@@ -106,21 +109,104 @@ public static synchronized void 
addSanitizeKeywords(String keywords) {
      * @param  uri The uri to sanitize.
      * @return     Returns null if the uri is null, otherwise the URI with the 
passphrase, password or secretKey
      *             sanitized.
-     * @see        #ALL_SECRETS and #USERINFO_PASSWORD for the matched pattern
+     * @see        SensitiveUtils#maskUserInfoCredentials(String, String) for 
how the userinfo password is found

Review Comment:
   💡 **Broken `@see` reference:** 
`SensitiveUtils#maskUserInfoCredentials(String, String)` doesn't exist on 
`camel-4.18.x` — only the package-private `maskUserInfo(String, String, 
boolean)` was ported (the public `maskUserInfoCredentials` wrapper stayed on 
`main`). This will produce a Javadoc warning.
   
   ```suggestion
        * @see        SensitiveUtils#maskUserInfo(String, String, boolean) for 
how the userinfo password is found
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to