[GitHub] [geronimo-jwt-auth] rmannibucau commented on a change in pull request #4: GERONIMO-6757 if we can avoid CDI.current by injecting, please do! otherwise go ahead.

[GitBox <gi...@apache.org>]

rmannibucau commented on a change in pull request #4: GERONIMO-6757 if we can avoid CDI.current by injecting, please do! otherwise go ahead.
URL: https://github.com/apache/geronimo-jwt-auth/pull/4#discussion_r355215239
 
 

 ##########
 File path: src/main/java/org/apache/geronimo/microprofile/impl/jwtauth/jwt/SignatureValidator.java
 ##########
 @@ -63,7 +63,7 @@ private void init() {
         jcaProvider = config.read("jca.provider", null);
     }
 
-    void verifySignature(final String alg, final String key, final String signingString, final String expected) {
+    public void verifySignature(final String alg, final String key, final String signingString, final String expected) {
 
 Review comment:
   Needed?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services