FrankChen021 commented on PR #19828: URL: https://github.com/apache/druid/pull/19828#issuecomment-5138149610
Follow-up on the remaining CodeQL alert: the declaration-level SuppressWarnings entry was not honored for this result location. CodeQL requires its source suppression comment on the line immediately before the alert, so I moved java/potentially-weak-cryptographic-algorithm there while retaining the separate InsecureCryptoUsage annotation for Error Prone. CryptoServiceTest passes 9/9, Maven validation succeeds, and git diff --check passes. Pushed as 26dd631697; this PR remains draft while the new current-head pipeline runs. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
