merrimanr commented on a change in pull request #1275: METRON-1878: Add Metron 
as a Knox service
URL: https://github.com/apache/metron/pull/1275#discussion_r243589215
 
 

 ##########
 File path: dependencies_with_url.csv
 ##########
 @@ -495,3 +495,6 @@ 
org.fusesource.jansi:jansi:jar:1.16:compile,ASLv2,https://github.com/fusesource/
 com.vividsolutions:jts:jar:1.13:compile
 joda-time:joda-time:jar:2.10:compile
 org.elasticsearch:securesm:jar:1.2:compile
+com.github.stephenc.jcip:jcip-annotations:jar:1.0-1:compile,ASLv2,http://stephenc.github.io/jcip-annotations/
 
 Review comment:
   The "nimbus-jose-jwt" dependency is the JSON web token implementation.  This 
is used to perform operations on the Knox token that is passed around the 
various services and UIs.
   
   I believe the "jcip-annotations" is included as a transitive dependency of 
the "nimbus-jose-jwt".  To be honest I'm not that familiar with JCIP 
annotations.  Should we try to exclude it?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to