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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0033665
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Retail Modules] Web POSminorhave not tried2016-08-09 23:012016-08-18 08:54
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisionbd9ac17f511a
ProjectionnoneETAnoneTarget VersionRR16Q4
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

0033665: 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
TagsNo tags attached.
Attached Filespng file icon 19-42-14 FAILED- CMGa10_CashManagementDeposit.png [^] (89,989 bytes) 2016-08-09 23:01


diff file icon 33665.diff [^] (4,579 bytes) 2016-08-11 18:30 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
has duplicate defect 0033745 closedRetail WEB POS login requests are executed serveral times in case of timeout 

-  Notes
(0089061)
hgbot (developer)
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 (developer)
2016-08-18 08:54

Tested and reviewed

- Issue History
Date Modified Username Field Change
2016-08-09 23:01 mtaal New Issue
2016-08-09 23:01 mtaal Assigned To => mtaal
2016-08-09 23:02 mtaal File Added: 19-42-14 FAILED- CMGa10_CashManagementDeposit.png
2016-08-09 23:02 mtaal Triggers an Emergency Pack => No
2016-08-11 18:30 mtaal File Added: 33665.diff
2016-08-12 00:25 hgbot Checkin
2016-08-12 00:25 hgbot Note Added: 0089061
2016-08-12 00:25 hgbot Status new => resolved
2016-08-12 00:25 hgbot Resolution open => fixed
2016-08-12 00:25 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/bd9ac17f511a648f51da299dabbdbdb6208c82b2 [^]
2016-08-16 10:28 mtaal Review Assigned To => migueldejuana
2016-08-18 08:54 migueldejuana Note Added: 0089198
2016-08-18 08:54 migueldejuana Status resolved => closed
2016-08-19 08:57 migueldejuana Relationship added has duplicate 0033745


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker