Merge pull request #28 from kuisathaverat/JENKINS-31612

[JENKINS-31612] Fix filter chain
jenkinsci · Dec 2, 2016 · cfa947c · cfa947c · daniel-beck · Dec 5, 2016
2 parents f31a787 + dc7808b
commit cfa947c
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/reverse_proxy_auth/ReverseProxySecurityRealm.java b/src/main/java/org/jenkinsci/plugins/reverse_proxy_auth/ReverseProxySecurityRealm.java
@@ -31,12 +31,9 @@
 import hudson.model.Descriptor;
 import hudson.model.Hudson;
 import hudson.model.User;
-import hudson.security.GroupDetails;
-import hudson.security.UserMayOrMayNotExistException;
+import hudson.security.*;
 import hudson.tasks.Mailer;
 import hudson.tasks.Mailer.UserProperty;
-import hudson.security.LDAPSecurityRealm;
-import hudson.security.SecurityRealm;
 import hudson.util.FormValidation;
 import hudson.util.Scrambler;
 import hudson.util.spring.BeanBuilder;
@@ -446,7 +443,7 @@ public String getLDAPURL() {
 
 	@Override
 	public Filter createFilter(FilterConfig filterConfig) {
-		return new Filter() {
+		Filter filter = new Filter() {
 			public void init(FilterConfig filterConfig) throws ServletException {
 			}
 
@@ -555,6 +552,8 @@ public void doFilter(ServletRequest request,
 			public void destroy() {
 			}
 		};
+		Filter defaultFilter = super.createFilter(filterConfig);
+		return new ChainedServletFilter(filter, defaultFilter);
 	}
 
 	@Override