2013/11/29 <[email protected]>: > Author: markt > Date: Fri Nov 29 19:23:13 2013 > New Revision: 1546630 > > URL: http://svn.apache.org/r1546630 > Log: > Remove unnecessary "throws ServletException" > > Modified: > tomcat/trunk/java/org/apache/catalina/Authenticator.java > tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java > > Modified: tomcat/trunk/java/org/apache/catalina/Authenticator.java > URL: > http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/Authenticator.java?rev=1546630&r1=1546629&r2=1546630&view=diff > ============================================================================== > --- tomcat/trunk/java/org/apache/catalina/Authenticator.java (original) > +++ tomcat/trunk/java/org/apache/catalina/Authenticator.java Fri Nov 29 > 19:23:13 2013 > @@ -51,5 +51,5 @@ public interface Authenticator { > public void login(String userName, String password, Request request) > throws ServletException; > > - public void logout(Request request) throws ServletException; > + public void logout(Request request);
This is an interface. Why custom authenticators aren't allowed to throw a ServletException here? > } > > Modified: > tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java > URL: > http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java?rev=1546630&r1=1546629&r2=1546630&view=diff > ============================================================================== > --- > tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java > (original) > +++ > tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java > Fri Nov 29 19:23:13 2013 > @@ -849,7 +849,7 @@ public abstract class AuthenticatorBase > } > > @Override > - public void logout(Request request) throws ServletException { > + public void logout(Request request) { > register(request, request.getResponse(), null, > null, null, null); > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
