abhishekagarwal87 commented on code in PR #16147:
URL: https://github.com/apache/druid/pull/16147#discussion_r1528100646


##########
owasp-dependency-check-suppressions.xml:
##########
@@ -664,4 +664,22 @@
     ]]></notes>
     <cve>CVE-2023-36415</cve>
   </suppress>
+  <suppress>
+    <!-- Covers DOS on identity server by triggering high resource 
consumption. Used in Azure as a client.
+    Current latest version of Azure BOM (1.2.21) still uses 9.30.2, whereas 
bug resolved in 9.37.3 -->

Review Comment:
   The reasoning is a bit cryptic. Do you mean to say that CVE only impacts the 
server but not the client? 



##########
extensions-core/druid-pac4j/pom.xml:
##########
@@ -38,7 +38,7 @@
 
     <!-- Following must be updated along with any updates to pac4j version. 
One can find the compatible version of nimbus libraries in org.pac4j:pac4j-oidc 
dependencies-->
     <nimbus.lang.tag.version>1.7</nimbus.lang.tag.version>
-    <nimbus.jose.jwt.version>8.22.1</nimbus.jose.jwt.version>
+    <nimbus.jose.jwt.version>9.37.2</nimbus.jose.jwt.version>

Review Comment:
   is it compatible with pac4j-oidc dependency currently in use? 



-- 
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]

Reply via email to