svn commit: r1760463 - /ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1760463 - /ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java

shijh
Author: shijh
Date: Tue Sep 13 01:02:06 2016
New Revision: 1760463

URL: http://svn.apache.org/viewvc?rev=1760463&view=rev
Log:
Fix for View screen in Apache Solr Integration Component is broken (OFBIZ-8225).

The duplicated login form is caused by not return after ContextFilter do the filter.

Modified:
    ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java

Modified: ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java?rev=1760463&r1=1760462&r2=1760463&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java (original)
+++ ofbiz/trunk/specialpurpose/solr/src/main/java/org/apache/ofbiz/solr/webapp/OFBizSolrContextFilter.java Tue Sep 13 01:02:06 2016
@@ -140,8 +140,7 @@ public class OFBizSolrContextFilter exte
         String servletPath = httpRequest.getServletPath();
         if (UtilValidate.isNotEmpty(servletPath) && servletPath.equals("/control")) {
          contextFilter.doFilter(httpRequest, httpResponse, chain);
-            // we're done checking; continue on
-            chain.doFilter(request, response);
+         return;
         } else {
             // check if the request is from an authorized user
             if (UtilValidate.isNotEmpty(servletPath) && (servletPath.startsWith("/admin/") || servletPath.endsWith("/update")