Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0037644 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Web POS | major | always | 2018-01-15 18:53 | 2018-01-16 12:04 | |||
| Reporter | marvintm | View Status | public | |||||
| Assigned To | marvintm | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | d32c1f1c4bcf | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | migueldejuana | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0037644: LogClient should not be sent if session is not valid | |||||||
| Description | Currently logclient is always sent, even if the WebPOS already detected that session is no longer valid, and that the user must log in again. This wastes network bandwidth, which can in some cases be problematic. Instead, the process should not be run if the session is not valid, and should only be restarted when the user logs in again. | |||||||
| Steps To Reproduce | - Log in the WebPOS, with the "Network" tab in developer tools open. - Restart tomcat (to force session lost) - Wait some time. - At some point, the WebPOS detects that the session is lost, and shows the popup that explains the user that he needs to log in again. - Wait more time. Check the "Network" tab, and realize that the log client request is still triggered periodically. This doesn't make sense, as it wastes bandwidth without any chance of succeeding. | |||||||
| Tags | Performance | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||||||||||||||||
|
||||||||||||||||||||||
 Relationships |
|
Notes |
|
|
(0101712) hgbot (developer) 2018-01-16 08:31 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: d32c1f1c4bcfd8b52c5e640627fb0d1381c57edf Author: Antonio Moreno <antonio.moreno <at> openbravo.com> Date: Mon Jan 15 18:56:46 2018 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d32c1f1c4bcfd8b52c5e640627fb0d1381c57edf [^] Fixed issue 37644. LogClient request will no longer happen if session was lost. --- M web/org.openbravo.mobile.core/source/component/ob-terminal-component.js M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js M web/org.openbravo.mobile.core/source/utils/logClientSyncUtils.js --- |
|
(0101717) hgbot (developer) 2018-01-16 12:01 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 4d5709f5f93d359f30cbfe978d3b726ed7c36502 Author: Antonio Moreno <antonio.moreno <at> openbravo.com> Date: Tue Jan 16 11:58:35 2018 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4d5709f5f93d359f30cbfe978d3b726ed7c36502 [^] Related to issue 37644. Added API to enable/disable log client process, with two main goals: -Disable the interval when session is lost, to remove unnecessary overhead -Try to prevent potential duplicate calls to the log client process --- M web/org.openbravo.mobile.core/source/component/ob-terminal-component.js M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js M web/org.openbravo.mobile.core/source/utils/logClientSyncUtils.js --- |
|
(0101718) migueldejuana (developer) 2018-01-16 12:04 |
Tested and reviewed |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2018-01-15 18:53 | marvintm | New Issue | |
| 2018-01-15 18:53 | marvintm | Assigned To | => Retail |
| 2018-01-15 18:53 | marvintm | Triggers an Emergency Pack | => No |
| 2018-01-15 18:53 | marvintm | Status | new => scheduled |
| 2018-01-15 18:53 | marvintm | Assigned To | Retail => marvintm |
| 2018-01-16 08:31 | hgbot | Checkin | |
| 2018-01-16 08:31 | hgbot | Note Added: 0101712 | |
| 2018-01-16 08:31 | hgbot | Status | scheduled => resolved |
| 2018-01-16 08:31 | hgbot | Resolution | open => fixed |
| 2018-01-16 08:31 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d32c1f1c4bcfd8b52c5e640627fb0d1381c57edf [^] |
| 2018-01-16 08:33 | marvintm | Resolution time | => 1517180400 |
| 2018-01-16 10:27 | shuehner | Tag Attached: Performance | |
| 2018-01-16 10:27 | shuehner | Issue Monitored: shuehner | |
| 2018-01-16 12:01 | hgbot | Checkin | |
| 2018-01-16 12:01 | hgbot | Note Added: 0101717 | |
| 2018-01-16 12:04 | migueldejuana | Review Assigned To | => migueldejuana |
| 2018-01-16 12:04 | migueldejuana | Note Added: 0101718 | |
| 2018-01-16 12:04 | migueldejuana | Status | resolved => closed |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |