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, and modifying existing one, this
means applications must use the version of this API in which the methods are
added/modified (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 ?
##########
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, and modifying existing ones, this
means applications must use the version of this API in which the methods are
added/modified (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]
