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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0034283
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] StoreServerminorhave not tried2016-10-24 15:552017-01-17 13:46
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in VersionRR16Q4.1
StatusclosedFix in branchFixed in SCM revision77f92549d3d0
ProjectionnoneETAnoneTarget VersionRR16Q4.1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034283: 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 ]
blocks defect 0034282RR17Q1 closedmtaal Support different protocols (http and https) for store and central server 

-  Notes
(0092115)
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
---
(0092117)
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
---
(0092133)
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
---
(0092135)
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
---
(0092173)
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
---
(0092175)
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
---
(0092177)
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
---
(0092179)
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
---
(0093014)
hgbot (developer)
2016-12-27 11:26

 Repository: retail/backports/3.0RR16Q4.1/org.openbravo.retail.posterminal
Changeset: b53c8a6fa29e597d18e6c92266d8d65847f50ea0
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Dec 27 11:26:32 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4.1/org.openbravo.retail.posterminal/rev/b53c8a6fa29e597d18e6c92266d8d65847f50ea0 [^]

Related to issue 34283: Support different protocols (http and https) for store and central server
Current server is set in MobileCoreLoginUtilsServlet, not needed to set in retail posterminal
login utils servlet

---
M src/org/openbravo/retail/posterminal/LoginUtilsServlet.java
---
(0093015)
hgbot (developer)
2016-12-27 11:28

 Repository: retail/backports/3.0RR16Q4.1/org.openbravo.mobile.core
Changeset: 77f92549d3d0b81fd68c23a9d5df3a3224fa2769
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Dec 27 11:27:53 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4.1/org.openbravo.mobile.core/rev/77f92549d3d0b81fd68c23a9d5df3a3224fa2769 [^]

Fixes issue 34283: 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
---

- Issue History
Date Modified Username Field Change
2016-10-24 15:55 mtaal Type defect => backport
2016-10-24 15:55 mtaal Target Version RR17Q1 => RR16Q4
2016-12-06 10:53 hgbot Checkin
2016-12-06 10:53 hgbot Note Added: 0092115
2016-12-06 10:53 hgbot Status scheduled => resolved
2016-12-06 10:53 hgbot Resolution open => fixed
2016-12-06 10:53 hgbot Fixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/9cf608fa0f7ee3543fb5b5fec4bd90c0c7675e8f [^]
2016-12-06 10:53 hgbot Checkin
2016-12-06 10:53 hgbot Note Added: 0092117
2016-12-06 10:53 hgbot Fixed in SCM revision http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/9cf608fa0f7ee3543fb5b5fec4bd90c0c7675e8f [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/a27a3351ef17e839c3953e9663391bd7a6e71587 [^]
2016-12-06 10:55 hgbot Checkin
2016-12-06 10:55 hgbot Note Added: 0092133
2016-12-06 10:55 hgbot Fixed in SCM revision http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/a27a3351ef17e839c3953e9663391bd7a6e71587 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/e3cdf7ed0bdd73391b681571e97cde589ee11073 [^]
2016-12-06 10:55 hgbot Checkin
2016-12-06 10:55 hgbot Note Added: 0092135
2016-12-06 10:55 hgbot Fixed in SCM revision http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/e3cdf7ed0bdd73391b681571e97cde589ee11073 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/0bcc4640b61abc1720376ff956e9ad63ed7d4d7b [^]
2016-12-07 12:00 mtaal Target Version RR16Q4 => RR16Q4.1
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092173
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092175
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092177
2016-12-07 12:05 hgbot Checkin
2016-12-07 12:05 hgbot Note Added: 0092179
2016-12-07 12:06 Orekaria Status resolved => new
2016-12-07 12:06 Orekaria Resolution fixed => open
2016-12-07 12:06 Orekaria Status new => scheduled
2016-12-27 11:23 mtaal Review Assigned To => marvintm
2016-12-27 11:26 hgbot Checkin
2016-12-27 11:26 hgbot Note Added: 0093014
2016-12-27 11:28 hgbot Checkin
2016-12-27 11:28 hgbot Note Added: 0093015
2016-12-27 11:28 hgbot Status scheduled => resolved
2016-12-27 11:28 hgbot Resolution open => fixed
2016-12-27 11:28 hgbot Fixed in SCM revision http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/0bcc4640b61abc1720376ff956e9ad63ed7d4d7b [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4.1/org.openbravo.mobile.core/rev/77f92549d3d0b81fd68c23a9d5df3a3224fa2769 [^]
2017-01-17 13:46 marvintm Status resolved => closed
2017-01-17 13:46 marvintm Fixed in Version => RR16Q4.1

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker