/[Apache-SVN]
ViewVC logotype

Revision 1784771


Jump to revision: Previous Next
Author: jleroux
Date: Tue Feb 28 16:10:45 2017 UTC (7 years, 2 months ago)
Changed paths: 3
Log Message:
Reverts revision(s) 1784745 from ofbiz/branches/release13.07:
"Partially backport the fix idea from trunk framework for revision: 1784549  " 
------------------------------------------------------------------------
r1784549 | jleroux | 2017-02-27 13:33:42 +0100 (lun. 27 fev. 2017) | 26 lignes

Here, unlike in r1784549, I kept getDefaultServerRootUrl() and did no changes in url.properties. URL properties will be, for instance, used to specify the ports used in demos (18443/18080)

Fixed: "Login and logout process in demos shows a certificate issue"
(OFBIZ-9206)

Also fixes "16.11 ofbizUrl include host+port and break some reverse-proxy/docker
 setups"
(OFBIZ-9224)

The idea is to use the port initially passed by a successful login request as it's done
since WebSiteProperties class exists. I also don't add twice the portOffset when it's build 
from the request, and only then. Keep it as is when it's build from a WebSite GenericValue. 
We then trust the user and don't rely on the request.

Thanks: Pierre Smits and Leonard Lin for reports
------------------------------------------------------------------------
........


Changed paths

Path Details
Directoryofbiz/branches/release13.07/framework/common/src/org/ofbiz/common/email/NotificationServices.java modified , text changed
Directoryofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/control/RequestHandler.java modified , text changed
Directoryofbiz/branches/release13.07/framework/webapp/src/org/ofbiz/webapp/ftl/OfbizUrlTransform.java modified , text changed

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26