jrihtarsic commented on code in PR #260:
URL: https://github.com/apache/ws-wss4j/pull/260#discussion_r1452985909


##########
ws-security-dom/src/main/java/org/apache/wss4j/dom/message/WSSecSignature.java:
##########
@@ -898,6 +906,50 @@ private X509Certificate[] getSigningCerts() throws 
WSSecurityException {
         return certs;
     }
 
+    /**
+     * The method returns EdDSA signature algorithm for public key type 
(Ed25519 or Ed448).
+     *
+     * @param publicKey the public key to get the algorithm from
+     * @return the siganture algorithm (ED25519 or ED448) for the EdDSA public 
key
+     * @throws WSSecurityException if the algorithm cannot be determined
+     */
+    public String getSigAlgorithmForEdDSAKey(PublicKey publicKey) throws 
WSSecurityException {

Review Comment:
   You are absolutely right. No need to be public. Now is fixed to private 
static. 



-- 
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: dev-unsubscr...@ws.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@ws.apache.org
For additional commands, e-mail: dev-h...@ws.apache.org

Reply via email to