michael-o commented on a change in pull request #225: BZ 63681: Introduce
RealmBase#authenticate(GSSName, GSSCredential) an…
URL: https://github.com/apache/tomcat/pull/225#discussion_r350608651
##########
File path: java/org/apache/catalina/Realm.java
##########
@@ -116,6 +118,17 @@ public Principal authenticate(String username, String
digest,
public Principal authenticate(GSSContext gssContext, boolean storeCreds);
+ /**
+ * Try to authenticate using a {@link GSSName}
+ *
+ * @param gssName The {@link GSSName} of the principal to look up
+ * @param gssCredential The {@link GSSCredential} of the principal, may be
+ * {@code null}
+ * @return the associated principal, or {@code null} if there is none
+ */
+ public Principal authenticate(GSSName gssName, GSSCredential
gssCredential);
+
Review comment:
OK, I see two alternatives here:
* Drop the change, move it to a change until Tomcat 10 emerges
* Introduce a `GSSRealm` interface which extends `Realm` and `ReamBase` will
implement `GSSRealm`. All Tomcat-provided authenticators will still use `Realm`
and custom authenticators can decide. I will also mark the interface as
deprecated for removal in Tomcat 10.
How does that sound? Which of those would you prefer?
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
