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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0034284
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] StoreServerminorhave not tried2016-10-24 15:552016-11-16 13:17
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisiond56eb00eb3f0
ProjectionnoneETAnoneTarget VersionRR16Q3.3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomigueldejuana
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034284: 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
(0090833)
hgbot (developer)
2016-10-25 00:11

Repository: retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core
Changeset: d56eb00eb3f0de2f8a97330b592a4bd413d56b33
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Oct 25 00:11:29 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/d56eb00eb3f0de2f8a97330b592a4bd413d56b33 [^]

Fixes issue 34284: Support different protocols (http and https) for store and central server
Do not strip off the http/https, only add it if needed in the client side code

---
M src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0091601)
migueldejuana (developer)
2016-11-16 13:17

Reviewed

- 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 => RR16Q3.3
2016-10-25 00:10 mtaal Review Assigned To => migueldejuana
2016-10-25 00:11 hgbot Checkin
2016-10-25 00:11 hgbot Note Added: 0090833
2016-10-25 00:11 hgbot Status scheduled => resolved
2016-10-25 00:11 hgbot Resolution open => fixed
2016-10-25 00:11 hgbot Fixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/d56eb00eb3f0de2f8a97330b592a4bd413d56b33 [^]
2016-11-16 13:17 migueldejuana Note Added: 0091601
2016-11-16 13:17 migueldejuana Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker