svn commit: r1626459 - in /ofbiz/branches/release13.07/framework/webapp: config/url.properties src/org/ofbiz/webapp/control/RequestHandler.java

[ashish-18]

Author: ashish
Date: Sat Sep 20 15:35:06 2014
New Revision: 1626459

URL: http://svn.apache.org/r1626459
Log:
Applied bug fix from trunk r1626458.
Applied bug fix from jira issue - OFBIZ-254 - Noisy, Possibly Unnecessary Logging when a browser request is cancelled by the user.
Thanks Jacopo for reporting the issue and thanks Pranay for the contribution.

Modified:
    ofbiz/branches/release13.07/framework/webapp/config/url.properties
    ofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java

Modified: ofbiz/branches/release13.07/framework/webapp/config/url.properties
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/framework/webapp/config/url.properties?rev=1626459&r1=1626458&r2=1626459&view=diff
==============================================================================
--- ofbiz/branches/release13.07/framework/webapp/config/url.properties (original)
+++ ofbiz/branches/release13.07/framework/webapp/config/url.properties Sat Sep 20 15:35:06 2014
@@ -22,11 +22,11 @@
 
 # HTTPS Port (Secure port)
 port.https.enabled=Y
-port.https=8443
+port.https=28443
 force.https.host=
 
 # HTTP Port (Not Secure port)
-port.http=8080
+port.http=28080
 force.http.host=
 
 # Static Content URLs to make it easy to move the serving load for static content to other machines

Modified: ofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java?rev=1626459&r1=1626458&r2=1626459&view=diff
==============================================================================
--- ofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java (original)
+++ ofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java Sat Sep 20 15:35:06 2014
@@ -924,14 +924,11 @@ public class RequestHandler {
         try {
             resp.flushBuffer();
         } catch (java.io.IOException e) {
-            /*If request is an ajax request and user calls abort() method for on ajax request then skip throwing of RequestHandlerException .
-             Specially its done for async ajax auto completer call, if we call abort() method on ajax request then its showing broken pipe exception on console,
-             because request is aborted by client (browser).*/
-            if (!"XMLHttpRequest".equals(req.getHeader("X-Requested-With"))) {
-                throw new RequestHandlerException("Error flushing response buffer", e);
-            } else {
-                if (Debug.verboseOn()) Debug.logVerbose("Skip Request Handler Exception for ajax request.", module);
-            }
+            /* If any request gets aborted before completing, i.e if a user requests a page and cancels that request before the page is rendered and returned
+               or if request is an ajax request and user calls abort() method for on ajax request then its showing broken pipe exception on console,
+               skip throwing of RequestHandlerException. JIRA Ticket - OFBIZ-254
+            */
+            if (Debug.verboseOn()) Debug.logVerbose("Skip Request Handler Exception that is caused due to aborted requests. " + e.getMessage(), module);
         }
 
         String vname = (String) req.getAttribute("_CURRENT_VIEW_");