Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0034282
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] StoreServerminorhave not tried2016-10-24 15:552016-12-07 12:05
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision8078590c67e8
ProjectionnoneETAnoneTarget VersionRR17Q1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034282: Support different protocols (http and https) for store and central server

DescriptionCurrently we don't support different protocols http and https for the central server. If for example the store has http:// and the central server has https:// then still WebPOS will try to call central with http://. [^] The central server will try to redirect webpos to https://. [^] However an error is thrown [1], this is not supported in cross domain pre-flight requests [2].

[1]
XMLHttpRequest cannot load http://testhipertextil.practicsbs.com/openbravo/org.openbravo.mobile.core.service.jsonrest/org.openbravo.retail.posterminal.ProcessCashMgmtMaster. [^] Response for preflight is invalid (redirect)

[2]
http://stackoverflow.com/questions/23412037/302-redirect-after-cors-preflight [^]
Steps To Reproduceinstall central and store
use https for central
http for store
login in to the store
stop store server
work offline on store server --> errors show up
Proposed SolutionDon't strip http/https from the url when sending to the client [1]
Check when setting the server in the request router code that the http/https is there. If so then don't prefix the location.protocol

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/6e967b9921b3/src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java#l206 [^]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/6e967b9921b3/web/org.openbravo.mobile.core/source/data/ob-requestrouter.js#l656 [^]



TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
depends on backport 0034283RR16Q4.1 closedmtaal Support different protocols (http and https) for store and central server 
depends on backport 0034284RR16Q3.3 closedmtaal Support different protocols (http and https) for store and central server 

-  Notes
(0090884)
hgbot (developer)
2016-10-26 08:29

 Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 34bc5e535958c57eb0ca26127f15095a76d060fe
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Oct 26 08:29:02 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/34bc5e535958c57eb0ca26127f15095a76d060fe [^]

Fixes issue 34282: Support different protocols (http and https) for store and central server
Don't take of http from server address when creating list of server jsons to be send to the client
On the client prefix/set the right protocol when the current server or not yet set
Determine in a better way if the server is a current server

---
M src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java
M src/org/openbravo/mobile/core/servercontroller/MobileServerController.java
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0090885)
hgbot (developer)
2016-10-26 08:30

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6804878de3c3b044a0afe14be40bc81c4afd30ea
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Oct 26 08:30:03 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6804878de3c3b044a0afe14be40bc81c4afd30ea [^]

Related to issue 34282: Support different protocols (http and https) for store and central server
Determine in a better/more performant way if the server definition is for the current server.

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0090967)
hgbot (developer)
2016-10-27 21:39

 Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 8078590c67e873ebd82d4caeb99ffc508746409b
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Oct 27 21:38:21 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/8078590c67e873ebd82d4caeb99ffc508746409b [^]

Fixes issue 34282: Support different protocols (http and https) for store and central server
Make sure that there is only one current server and also use the correct part of the url to set the currentserver
if none was defined on the server

---
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0090968)
hgbot (developer)
2016-10-27 21:40

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: c075a072c539637a2c1f3dbb4618e5f25c3f7f8f
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Oct 27 21:40:12 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c075a072c539637a2c1f3dbb4618e5f25c3f7f8f [^]

Related to issue 34282: Support different protocols (http and https) for store and central server
Partially rollback earlier changeset, only keep the part related to selecting the correct
multi-server

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0091600)
migueldejuana (viewer)
2016-11-16 13:17

 Reviewed
(0092114)
hgbot (developer)
2016-12-06 10:53

 Repository: retail/backports/3.0RR16Q4/org.openbravo.mobile.core
Changeset: 9cf608fa0f7ee3543fb5b5fec4bd90c0c7675e8f
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Oct 26 08:29:02 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/9cf608fa0f7ee3543fb5b5fec4bd90c0c7675e8f [^]

Fixed issue 34283, backport of issue 34282: Support different protocols (http and https) for store and central server
Don't take of http from server address when creating list of server jsons to be send to the client
On the client prefix/set the right protocol when the current server or not yet set
Determine in a better way if the server is a current server

---
M src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java
M src/org/openbravo/mobile/core/servercontroller/MobileServerController.java
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0092116)
hgbot (developer)
2016-12-06 10:53

 Repository: retail/backports/3.0RR16Q4/org.openbravo.mobile.core
Changeset: a27a3351ef17e839c3953e9663391bd7a6e71587
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Oct 27 21:38:21 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/a27a3351ef17e839c3953e9663391bd7a6e71587 [^]

Fixes issue 34283, backport of issue 34282: Support different protocols (http and https) for store and central server
Make sure that there is only one current server and also use the correct part of the url to set the currentserver
if none was defined on the server

---
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0092132)
hgbot (developer)
2016-12-06 10:55

 Repository: retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal
Changeset: e3cdf7ed0bdd73391b681571e97cde589ee11073
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Oct 26 08:30:03 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/e3cdf7ed0bdd73391b681571e97cde589ee11073 [^]

Fixes issue 34283, backport of issue 34282: Support different protocols (http and https) for store and central server
Determine in a better/more performant way if the server definition is for the current server.

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0092134)
hgbot (developer)
2016-12-06 10:55

 Repository: retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal
Changeset: 0bcc4640b61abc1720376ff956e9ad63ed7d4d7b
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Oct 27 21:40:12 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/0bcc4640b61abc1720376ff956e9ad63ed7d4d7b [^]

Fixes issue 34283, backport of issue 34282: Support different protocols (http and https) for store and central server
Partially rollback earlier changeset, only keep the part related to selecting the correct
multi-server

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0092174)
hgbot (developer)
2016-12-07 12:05

 Repository: retail/backports/3.0RR16Q4/org.openbravo.mobile.core
Changeset: 2dd544569b1e6ec89e7d4b2af5e4727c106498b8
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Wed Dec 07 12:04:14 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/2dd544569b1e6ec89e7d4b2af5e4727c106498b8 [^]

Backed out changeset: a27a3351ef17; issue 34283, backport of issue 34282

---
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0092176)
hgbot (developer)
2016-12-07 12:05

 Repository: retail/backports/3.0RR16Q4/org.openbravo.mobile.core
Changeset: 24c8a0236915f5fcd950d6f6ef3175bdaeda08b6
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Wed Dec 07 12:04:20 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/24c8a0236915f5fcd950d6f6ef3175bdaeda08b6 [^]

Backed out changeset: 9cf608fa0f7e; issue 34283, backport of issue 34282

---
M src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java
M src/org/openbravo/mobile/core/servercontroller/MobileServerController.java
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0092178)
hgbot (developer)
2016-12-07 12:05

 Repository: retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal
Changeset: 51bfa1f6797b1688afd0e225f47deb3598827348
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Wed Dec 07 12:03:49 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/51bfa1f6797b1688afd0e225f47deb3598827348 [^]

Backed out changeset: 0bcc4640b61a; issue 34283, backport of issue 34282

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0092180)
hgbot (developer)
2016-12-07 12:05

 Repository: retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal
Changeset: ea84cef08404d79b57589d09507897168a752382
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Wed Dec 07 12:03:59 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/ea84cef08404d79b57589d09507897168a752382 [^]

Backed out changeset: e3cdf7ed0bdd; issue 34283, backport of issue 34282

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---

- Issue History
Date Modified Username Field Change
2016-10-24 15:55 mtaal New Issue
2016-10-24 15:55 mtaal Assigned To => mtaal
2016-10-24 15:55 mtaal OBNetwork customer => No
2016-10-24 15:55 mtaal Triggers an Emergency Pack => No
2016-10-24 15:55 mtaal Status new => scheduled
2016-10-26 08:29 hgbot Checkin
2016-10-26 08:29 hgbot Note Added: 0090884
2016-10-26 08:29 hgbot Status scheduled => resolved
2016-10-26 08:29 hgbot Resolution open => fixed
2016-10-26 08:29 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/34bc5e535958c57eb0ca26127f15095a76d060fe [^]
2016-10-26 08:30 hgbot Checkin
2016-10-26 08:30 hgbot Note Added: 0090885
2016-10-26 09:52 mtaal Status resolved => new
2016-10-26 09:52 mtaal Resolution fixed => open
2016-10-27 21:39 hgbot Checkin
2016-10-27 21:39 hgbot Note Added: 0090967
2016-10-27 21:39 hgbot Status new => resolved
2016-10-27 21:39 hgbot Resolution open => fixed
2016-10-27 21:39 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/34bc5e535958c57eb0ca26127f15095a76d060fe [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/8078590c67e873ebd82d4caeb99ffc508746409b [^]
2016-10-27 21:40 hgbot Checkin
2016-10-27 21:40 hgbot Note Added: 0090968
2016-11-07 17:00 mtaal Review Assigned To => migueldejuana
2016-11-16 13:17 migueldejuana Note Added: 0091600
2016-11-16 13:17 migueldejuana Status resolved => closed
2016-12-06 10:53 hgbot Checkin
2016-12-06 10:53 hgbot Note Added: 0092114
2016-12-06 10:53 hgbot Checkin
2016-12-06 10:53 hgbot Note Added: 0092116
2016-12-06 10:55 hgbot Checkin
2016-12-06 10:55 hgbot Note Added: 0092132
2016-12-06 10:55 hgbot Checkin
2016-12-06 10:55 hgbot Note Added: 0092134
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092174
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092176
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092178
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092180

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker