Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0024795Openbravo ERPA. Platformpublic2013-09-19 13:132022-02-01 08:05
adrianromero 
Triage Platform Base 
normalminorhave not tried
acknowledgedopen 
5
 
 
Core
No
0024795: If Openbravo is behind a proxy server that uses https. Login page is forwarded with protocol http.
In the case there is a proxy server that exposes the Openbravo application using the https protocol, but request to Openbravo from the proxy uses http, then during the login process Openbravo sends a redirection to the menu with the protocol http and sending the username and password clear through the net.

This is because when redirecting to menu, Openbravo builds an absolute URL using the request that is http not https
In description
Instead of building an absolute URL, build a relative URL in redirections.
Attached a fix for the case of the redirection in the login process only.
No tags attached.
patch Fix-24795.patch (4,723) 2013-09-19 13:14
https://issues.openbravo.com/file_download.php?file_id=6418&type=bug
Issue History
2013-09-19 13:13adrianromeroNew Issue
2013-09-19 13:13adrianromeroAssigned To => AugustoMauch
2013-09-19 13:13adrianromeroModules => Core
2013-09-19 13:13adrianromeroTriggers an Emergency Pack => No
2013-09-19 13:14adrianromeroFile Added: Fix-24795.patch
2013-09-19 13:30shuehnerIssue Monitored: shuehner
2015-03-17 14:37alostaleAssigned ToAugustoMauch => platform
2015-11-11 11:30alostalePriorityurgent => normal
2015-11-11 11:30alostaleSeveritymajor => minor
2015-11-11 11:30alostaleStatusnew => acknowledged
2015-11-11 13:14shuehnerNote Added: 0081704
2022-02-01 08:05alostaleAssigned Toplatform => Triage Platform Base

Notes
(0081704)
shuehner   
2015-11-11 13:14   
Note: there is probably server config bug involved.

As when having SSL termination done not in tomcat itself but in some other software before.

You need to configure that software properly to tell tomcat that the outside global url is with ssl.
When done correctly Openbravo will work just fine (ie. running live like that in all of ondemand).

Note:
That does not invalidate this issue, as not relying on this in some code make it more robust against that wrong configuration problem.