Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033665Retail ModulesWeb POSpublic2016-08-09 23:012016-08-18 08:54
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q4Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033665: POSLoginHandler call has a timeout which is too short and should not retry
DescriptionThe login request is a http request to the POSLoginHandler service. This service has a timeout which is too short. Therefore it results in the request router [2] trying to retry the request. If the request is retried and the first request already successfully logged in on the server then a message is displayed (see attached)


[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/052f0d3e3b18/web/org.openbravo.mobile.core/source/model/ob-terminal-model.js#l1863 [^]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/052f0d3e3b18/web/org.openbravo.mobile.core/source/data/ob-requestrouter.js#l127 [^]
Steps To ReproduceRun retail
Put breakpoint in POSLoginHandler servlet
Login to webpos
Check the chrome console
At the second request let the flow in java continue/not stop
Proposed Solution- increase the timeout to 60 seconds
- optionally pass the noRetries property in the ajaxrequest

The first option seems better
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 0033745 closed Retail WEB POS login requests are executed serveral times in case of timeout 
Attached Filespng 19-42-14 FAILED- CMGa10_CashManagementDeposit.png (89,989) 2016-08-09 23:01
https://issues.openbravo.com/file_download.php?file_id=9703&type=bug
png

diff 33665.diff (4,579) 2016-08-11 18:30
https://issues.openbravo.com/file_download.php?file_id=9709&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-08-09 23:01mtaalNew Issue
2016-08-09 23:01mtaalAssigned To => mtaal
2016-08-09 23:02mtaalFile Added: 19-42-14 FAILED- CMGa10_CashManagementDeposit.png
2016-08-09 23:02mtaalOBNetwork customer => No
2016-08-09 23:02mtaalTriggers an Emergency Pack => No
2016-08-11 18:30mtaalFile Added: 33665.diff
2016-08-12 00:25hgbotCheckin
2016-08-12 00:25hgbotNote Added: 0089061
2016-08-12 00:25hgbotStatusnew => resolved
2016-08-12 00:25hgbotResolutionopen => fixed
2016-08-12 00:25hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/bd9ac17f511a648f51da299dabbdbdb6208c82b2 [^]
2016-08-16 10:28mtaalReview Assigned To => migueldejuana
2016-08-18 08:54migueldejuanaNote Added: 0089198
2016-08-18 08:54migueldejuanaStatusresolved => closed
2016-08-19 08:57migueldejuanaRelationship addedhas duplicate 0033745

Notes
(0089061)
hgbot   
2016-08-12 00:25   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: bd9ac17f511a648f51da299dabbdbdb6208c82b2
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Fri Aug 12 00:25:00 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/bd9ac17f511a648f51da299dabbdbdb6208c82b2 [^]

Fixes issue 33665: POSLoginHandler call has a timeout which is too short and should not retry
Add no retries and longer timeout to request properties

---
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---
(0089198)
migueldejuana   
2016-08-18 08:54   
Tested and reviewed