Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034267Openbravo ERPA. Platformpublic2016-10-21 11:102016-12-16 18:38
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionno change required 
PlatformOS5OS Version
Product Version 
Target Version3.0PR17Q1Fixed in Version 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034267: Let the basekernelservlet handle cors requests
DescriptionMany servlets in retail support cors [1]. The base kernel servlet was missing it. This is however needed for mobile warehouse module which also uses direct datasource requests.

[1]
https://en.wikipedia.org/wiki/Cross-origin_resource_sharing [^]
Steps To ReproduceSet the cors headers in the request

 OBMOBCUtils.setCORSHeaders(request, response);
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0034063 closed mtaal Retail Modules [STORE SERVER 1791]Support offline and multi-server for Mobile Warehouse 
related to feature request 00343313.0PR17Q1 closed mtaal Openbravo ERP Support pre-defined allowed domains for cross-domain requests in a multi-server environment 
blocks feature request 0034298RR17Q1 closed mtaal Retail Modules Make use of common method to generate cors response headers 
blocks design defect 0034330RR17Q1 closed mtaal Retail Modules Support multi-server requests in a better more secure way 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-10-21 11:10mtaalNew Issue
2016-10-21 11:10mtaalAssigned To => mtaal
2016-10-21 11:10mtaalModules => Core
2016-10-21 11:10mtaalTriggers an Emergency Pack => No
2016-10-21 20:08mtaalReview Assigned To => alostale
2016-10-21 20:08mtaalRelationship addedrelated to 0034063
2016-10-23 10:37hgbotCheckin
2016-10-23 10:37hgbotNote Added: 0090784
2016-10-23 10:37hgbotStatusnew => resolved
2016-10-23 10:37hgbotResolutionopen => fixed
2016-10-23 10:37hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/27cdaf759fc805e141848ddf17d597ed4a5b2440 [^]
2016-10-23 11:01hgbotCheckin
2016-10-23 11:01hgbotNote Added: 0090786
2016-10-23 11:02mtaalStatusresolved => new
2016-10-23 11:02mtaalResolutionfixed => open
2016-10-23 16:10hgbotCheckin
2016-10-23 16:10hgbotNote Added: 0090795
2016-10-23 16:10hgbotStatusnew => resolved
2016-10-23 16:10hgbotResolutionopen => fixed
2016-10-23 16:10hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/27cdaf759fc805e141848ddf17d597ed4a5b2440 [^] => http://code.openbravo.com/erp/devel/pi/rev/e2e7fb70d085771ea0c714dcd9f547b05a1e85e0 [^]
2016-10-26 08:49mtaalRelationship addedblocks 0034298
2016-10-28 10:20alostaleNote Added: 0090984
2016-10-28 10:20alostaleStatusresolved => new
2016-10-28 10:20alostaleResolutionfixed => open
2016-10-28 10:45mtaalRelationship addedrelated to 0034331
2016-10-28 10:46mtaalRelationship addedblocks 0034330
2016-10-31 09:41hgbotCheckin
2016-10-31 09:41hgbotNote Added: 0091017
2016-10-31 10:39hgbotCheckin
2016-10-31 10:39hgbotNote Added: 0091019
2016-10-31 10:39hgbotStatusnew => resolved
2016-10-31 10:39hgbotResolutionopen => fixed
2016-10-31 10:39hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/e2e7fb70d085771ea0c714dcd9f547b05a1e85e0 [^] => http://code.openbravo.com/erp/devel/api-checks/rev/7467958c769061e8c4ecb410c177d3dfa06a3ea6 [^]
2016-11-04 08:09alostaleStatusresolved => closed
2016-11-04 08:10alostaleNote Added: 0091158
2016-11-04 08:10alostaleResolutionfixed => no change required
2016-12-16 18:38hudsonbotCheckin
2016-12-16 18:38hudsonbotNote Added: 0092544
2016-12-16 18:38hudsonbotCheckin
2016-12-16 18:38hudsonbotNote Added: 0092546
2016-12-16 18:38hudsonbotCheckin
2016-12-16 18:38hudsonbotNote Added: 0092547
2016-12-16 18:38hudsonbotCheckin
2016-12-16 18:38hudsonbotNote Added: 0092581

Notes
(0090784)
hgbot   
2016-10-23 10:37   
Repository: erp/devel/pi
Changeset: 27cdaf759fc805e141848ddf17d597ed4a5b2440
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun Oct 23 10:36:41 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/27cdaf759fc805e141848ddf17d597ed4a5b2440 [^]

Fixes issue 34267: Let the basekernelservlet handle cors requests
Set the cors headers when serving a request

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/BaseKernelServlet.java
---
(0090786)
hgbot   
2016-10-23 11:01   
Repository: erp/devel/pi
Changeset: b61ce156e6a025d7b3d4833c0d8883210be5c729
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun Oct 23 11:00:57 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/b61ce156e6a025d7b3d4833c0d8883210be5c729 [^]

Related to issue 34267: Let the basekernelservlet handle cors requests
Backout of changeset as unintended dependency on mobile core was created.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/BaseKernelServlet.java
---
(0090795)
hgbot   
2016-10-23 16:10   
Repository: erp/devel/pi
Changeset: e2e7fb70d085771ea0c714dcd9f547b05a1e85e0
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun Oct 23 16:10:33 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/e2e7fb70d085771ea0c714dcd9f547b05a1e85e0 [^]

Fixes issue 34267: Let the basekernelservlet handle cors requests
Set cors headers in the basekernelservlet

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/BaseKernelServlet.java
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelUtils.java
---
(0090984)
alostale   
2016-10-28 10:20   
Reopening after discussion with MTA
(0091017)
hgbot   
2016-10-31 09:41   
Repository: erp/devel/pi
Changeset: b06a2c5097c19c5cd019c7462f3d83c3c092765e
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Oct 31 09:41:22 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b06a2c5097c19c5cd019c7462f3d83c3c092765e [^]

Related to issue 34267: Let the basekernelservlet handle cors requests
Backout of changeset

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/BaseKernelServlet.java
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelUtils.java
---
(0091019)
hgbot   
2016-10-31 10:39   
Repository: erp/devel/api-checks
Changeset: 7467958c769061e8c4ecb410c177d3dfa06a3ea6
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Oct 31 10:38:58 2016 +0100
URL: http://code.openbravo.com/erp/devel/api-checks/rev/7467958c769061e8c4ecb410c177d3dfa06a3ea6 [^]

Fixes issue 34267: Let the basekernelservlet handle cors requests
Correct api baseline for remove of method which was only temporarily there

---
M java/reference/java.japi.gz
---
(0091158)
alostale   
2016-11-04 08:10   
Closed with no change, proper solution will be implemented by 0034331
(0092544)
hudsonbot   
2016-12-16 18:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/dc8bf00badd0 [^]
Maturity status: Test
(0092546)
hudsonbot   
2016-12-16 18:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/dc8bf00badd0 [^]
Maturity status: Test
(0092547)
hudsonbot   
2016-12-16 18:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/dc8bf00badd0 [^]
Maturity status: Test
(0092581)
hudsonbot   
2016-12-16 18:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/dc8bf00badd0 [^]
Maturity status: Test