opendev/gerrit
Code Issues Proposed changes

Allow logging in at /login

Browse Source 
Prior to this change, login was only /login/ and logout was only
/logout. We don't need to serve /logout/ necessarily, but we should
serve /login for at least some consistency.

Change-Id: I3f1b7e84f20187548cd9cf905cc2631350f7cc1a
This commit is contained in:
Dave Borowitz
2013-03-28 13:56:49 -04:00
parent 63befc1cc8
commit a9e42e15d2
5 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-httpd/src/main/java/com/google/gerrit/httpd/auth/become/BecomeAnyAccountModule.java
View File

@@ -19,6 +19,6 @@ import com.google.inject.servlet.ServletModule;
public class BecomeAnyAccountModule extends ServletModule { public class BecomeAnyAccountModule extends ServletModule {
@Override @Override
protected void configureServlets() { protected void configureServlets() {
serve("/login/*").with(BecomeAnyAccountLoginServlet.class); serve("/login", "/login/*").with(BecomeAnyAccountLoginServlet.class);
} }
} }

2
gerrit-httpd/src/main/java/com/google/gerrit/httpd/auth/container/HttpAuthModule.java
View File

@@ -21,6 +21,6 @@ public class HttpAuthModule extends ServletModule {
@Override @Override
protected void configureServlets() { protected void configureServlets() {
filter("/").through(HttpAuthFilter.class); filter("/").through(HttpAuthFilter.class);
serve("/login/*").with(HttpLoginServlet.class); serve("/login", "/login/*").with(HttpLoginServlet.class);
} }
} }

2
gerrit-httpd/src/main/java/com/google/gerrit/httpd/auth/container/HttpsClientSslCertModule.java
View File

@@ -21,6 +21,6 @@ public class HttpsClientSslCertModule extends ServletModule {
@Override @Override
protected void configureServlets() { protected void configureServlets() {
filter("/").through(HttpsClientSslCertAuthFilter.class); filter("/").through(HttpsClientSslCertAuthFilter.class);
serve("/login/*").with(HttpsClientSslCertLoginServlet.class); serve("/login", "/login/*").with(HttpsClientSslCertLoginServlet.class);
} }
} }

2
gerrit-httpd/src/main/java/com/google/gerrit/httpd/auth/ldap/LdapAuthModule.java
View File

@@ -20,6 +20,6 @@ import com.google.inject.servlet.ServletModule;
public class LdapAuthModule extends ServletModule { public class LdapAuthModule extends ServletModule {
@Override @Override
protected void configureServlets() { protected void configureServlets() {
serve("/login/*").with(LdapLoginServlet.class); serve("/login", "/login/*").with(LdapLoginServlet.class);
} }
} }

2
gerrit-openid/src/main/java/com/google/gerrit/httpd/auth/openid/OpenIdModule.java
View File

@@ -20,7 +20,7 @@ import com.google.inject.servlet.ServletModule;
public class OpenIdModule extends ServletModule { public class OpenIdModule extends ServletModule {
@Override @Override
protected void configureServlets() { protected void configureServlets() {
serve("/login/*").with(LoginForm.class); serve("/login", "/login/*").with(LoginForm.class);
serve("/" + OpenIdServiceImpl.RETURN_URL).with(OpenIdLoginServlet.class); serve("/" + OpenIdServiceImpl.RETURN_URL).with(OpenIdLoginServlet.class);
serve("/" + XrdsServlet.LOCATION).with(XrdsServlet.class); serve("/" + XrdsServlet.LOCATION).with(XrdsServlet.class);
filter("/").through(XrdsFilter.class); filter("/").through(XrdsFilter.class);