seanjmullan commented on code in PR #234:
URL: 
https://github.com/apache/santuario-xml-security-java/pull/234#discussion_r1410793725


##########
src/main/java/org/apache/xml/security/encryption/AgreementMethod.java:
##########
@@ -88,6 +91,22 @@ public interface AgreementMethod {
      */
     void setKANonce(byte[] kanonce);
 
+
+    /**
+     * Returns KeyDerivationMethod information used in the 
<code>AgreementMethod</code>.
+     * @return The KeyDerivationMethod information regarding the 
<code>AgreementMethod</code>.
+     */
+    KeyDerivationMethod getKeyDerivationMethod() throws XMLSecurityException;

Review Comment:
   By adding new methods to an interface, this means applications must use the 
version of this API in which the methods are added (they can't expect to use 
older versions w/o the methods). I guess that's ok, but not sure what our 
policy is on that - does this suggest we need a new major version when this is 
released,  @coheigea ? 



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

Reply via email to