mike-jumper commented on code in PR #846:
URL: https://github.com/apache/guacamole-client/pull/846#discussion_r1179613714
##########
guacamole-ext/src/main/java/org/apache/guacamole/net/auth/AuthenticationProvider.java:
##########
@@ -247,5 +247,42 @@ UserContext redecorate(UserContext decorated, UserContext
context,
* shutting down.
*/
void shutdown();
+
+ /**
+ * Called when the overall authentication attempt failed, despite a
+ * successful authentication or decoration for this authentication
provider.
+ * Implementations may add custom behavior here - by default this method
+ * does not do anything.
+ *
+ * @param authenticatedUser
+ * The user who successfully authenticated with this authentication
+ * provider.
+ *
+ * @param credentials
+ * The credentials which were most recently submitted for the given
+ * AuthenticatedUser. These are not guaranteed to be the same as the
+ * credentials associated with the AuthenticatedUser object, which are
+ * the credentials provided when the user originally authenticated.
+ */
+ default void notifyAuthenticationFailure(
+ AuthenticatedUser authenticatedUser, Credentials credentials)
{};
+
+ /**
+ * Called when the overall authentication attempt succeeded.
+ * Implementations may add custom behavior here - by default this method
+ * does not do anything.
+ *
+ * @param authenticatedUser
+ * The user who successfully authenticated with this authentication
+ * provider.
+ *
+ * @param credentials
+ * The credentials which were most recently submitted for the given
+ * AuthenticatedUser. These are not guaranteed to be the same as the
+ * credentials associated with the AuthenticatedUser object, which are
+ * the credentials provided when the user originally authenticated.
+ */
+ default void notifyAuthenticationSuccess(
+ AuthenticatedUser authenticatedUser, Credentials credentials)
{};
Review Comment:
Why this vs. the existing `AuthenticationSuccessEvent` and
`AuthenticationFailureEvent`?
--
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]
