You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2015/06/26 09:29:01 UTC

svn commit: r1687688 - /tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/JaspicAuthenticator.java

Author: markt
Date: Fri Jun 26 07:29:01 2015
New Revision: 1687688

URL: http://svn.apache.org/r1687688
Log:
Add a TODO

Modified:
    tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/JaspicAuthenticator.java

Modified: tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/JaspicAuthenticator.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/JaspicAuthenticator.java?rev=1687688&r1=1687687&r2=1687688&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/JaspicAuthenticator.java (original)
+++ tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/JaspicAuthenticator.java Fri Jun 26 07:29:01 2015
@@ -84,6 +84,9 @@ public class JaspicAuthenticator extends
         AuthStatus authStatus;
         Subject subject = new Subject();
         try {
+            // TODO: A number of the method calls below are synchronised. For
+            //       something that may get called on every request that is a
+            //       potential bottleneck.
             ServerAuthConfig authConfig = configProvider.getServerAuthConfig(MESSAGE_LAYER,
                     appContext, callbackHandler);
             String messageAuthContextId = authConfig.getAuthContextID(messageInfo);



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org