Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0046312 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [Retail Modules] Web POS | minor | have not tried | 2021-04-20 08:04 | 2021-04-21 10:13 | |||
| Reporter | caristu | View Status | public | |||||
| Assigned To | caristu | |||||||
| Priority | high | Resolution | fixed | Fixed in Version | RR21Q2 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | RR21Q2.1 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Regression level | Packaging and release | |||||||
| Regression date | 2021-03-04 | |||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/59e0d5edb148d43136aa6c37efcfa57d033828f7 [^] | |||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0046312: Context change is not properly checked in some requests | |||||||
| Description | With every backend request, it is checked whether the current context (client, org, user, role) has changed. If changed, a confirmation is shown to the user in order to log in again. This check is being wrongly skipped in some backend requests. | |||||||
| Steps To Reproduce | 1) Put a breakpoint here[1]. This is the place where the context check is skipped when it is not possible to get the information. 2) Log in in WebPOS. 3) Note that we are skipping the check in some requests although they contain context information [1] https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/blob/master/web/org.openbravo.mobile.core/app/integration/remote-server/BackendServer.js#L72 [^] | |||||||
| Proposed Solution | The context info is not being recovered properly in some requests, this is because we are recovering it from the "contextInfo" property of the response object and in some requests it comes inside "response.contextInfo" property instead | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0127401) hgbot (developer) 2021-04-20 18:11 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/215 [^] |
|
(0127407) hgbot (developer) 2021-04-21 10:13 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/215 [^] |
|
(0127408) hgbot (developer) 2021-04-21 10:13 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^] Changeset: 895082891e7c2a289e4cc529a65d22654ce2843c Author: Carlos Aristu <carlos.aristu@openbravo.com> Date: 2021-04-20T18:10:20+02:00 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/895082891e7c2a289e4cc529a65d22654ce2843c [^] fixes ISSUE-46312: context is not properly checked in some requests --- M web/org.openbravo.mobile.core/app/integration/remote-server/BackendServer.js --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2021-04-20 18:07 | caristu | Type | defect => backport |
| 2021-04-20 18:07 | caristu | Target Version | => RR21Q2.1 |
| 2021-04-20 18:11 | hgbot | Note Added: 0127401 | |
| 2021-04-21 10:12 | alostale | Assigned To | Retail => caristu |
| 2021-04-21 10:13 | hgbot | Resolution | open => fixed |
| 2021-04-21 10:13 | hgbot | Status | scheduled => closed |
| 2021-04-21 10:13 | hgbot | Note Added: 0127407 | |
| 2021-04-21 10:13 | hgbot | Fixed in Version | => RR21Q2 |
| 2021-04-21 10:13 | hgbot | Note Added: 0127408 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |