You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2012/10/06 13:00:15 UTC
svn commit: r1394996 - in /ofbiz/branches/release12.04: ./
framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java
Author: jleroux
Date: Sat Oct 6 11:00:14 2012
New Revision: 1394996
URL: http://svn.apache.org/viewvc?rev=1394996&view=rev
Log:
"Applied fix from trunk for revision: 1394995"
------------------------------------------------------------------------
r1394995 | jleroux | 2012-10-06 12:59:12 +0200 (sam., 06 oct. 2012) | 9 lines
A modified patch from Jyoti Sharma https://issues.apache.org/jira/browse/OFBIZ-5043
When i worked with Url Rewrite Filter, i faced the following issue:-
In case of http protocol in the url, https links on th epage were not being rewritten and when their was https url, http links were not rewritten.
The issue behind this what i found is that in RequestHandler.java encodeUrl() is not called in the above given case.
So, if we place response.encodeUrl() their the issue is resolved.
jleroux: This makes sense since we are still in the "if (encode) {" block. I just noticed that response != null was checked above so I placed the same check before encoding
------------------------------------------------------------------------
Modified:
ofbiz/branches/release12.04/ (props changed)
ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java
Propchange: ofbiz/branches/release12.04/
------------------------------------------------------------------------------
Merged /ofbiz/trunk:r1394995
Modified: ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java?rev=1394996&r1=1394995&r2=1394996&view=diff
==============================================================================
--- ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java (original)
+++ ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java Sat Oct 6 11:00:14 2012
@@ -1123,8 +1123,11 @@ public class RequestHandler {
newURL.insert(questionIndex, sessionId);
}
}
-
- encodedUrl = newURL.toString();
+ if (response != null) {
+ encodedUrl = response.encodeURL(newURL.toString());
+ } else {
+ encodedUrl = newURL.toString();
+ }
}
} else {
encodedUrl = newURL.toString();