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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0044490
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSminorhave not tried2020-06-25 14:202020-12-17 03:32
ReportershuehnerView Statuspublic 
Assigned Toranjith_qualiantech_com 
PrioritynormalResolutionfixedFixed in VersionRR20Q4
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0044490: Remove overhead in URL's of authenticationClient & authenticationToken (multiserver related)

DescriptionThose 2 values in request are part of code added to multiserver/storeserver to allow cross-server validation of requests/session.

However it seems to have a couple of bugs in practice
- Request are seen in which the 2 values are added multiple times in a single request (seen repeated up to 21 times)
For example seen in livebuild
/retail_modules_pgsql_pi/org.openbravo.mobile.core.checkServerA
vailability?authenticationClient=39363B0921BB4293B48383844325E84C&authenticationToken=<token>&authenticationClient=39363B0921BB4293B483
83844325E84C&authenticationToken=<token> now repeated 21 times

- Additionally those were only added to use in multi-server setup and should not be needed in single server install

Checking example big customer (19Q3)
- ca. 30% of GET URL length is only for those parameters (single copy of the values)
- 1.1M of 1.2M daily requests did have the token

As that causes some non-trivial overhead for (upload bandwidth, log-file volume, extra work for log-file parsing) maybe the feature should be completely disabled .
Steps To ReproduceConcrete steps how to get WebPOS to add those are not known.
However checking logs of livebuilds it seems quite typical to happen every day during usage
Proposed Solutiona.) Double check that those values are not useful with single-server setup
b.) Disable / remove the feature as being obsolete
TagsNo tags attached.
Attached Filesdiff file icon 44490_20Q1_mobilecore.diff [^] (15,239 bytes) 2020-12-17 03:32 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0122107)
hgbot (developer)
2020-08-18 03:51

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/51 [^]
(0122120)
hgbot (developer)
2020-08-18 08:28

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/51 [^]
(0122121)
hgbot (developer)
2020-08-18 08:28

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: fbd5c4207b0cf6fa5ed4170b158089547f84ffd3
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 2020-08-18T11:37:57+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/fbd5c4207b0cf6fa5ed4170b158089547f84ffd3 [^]

Fixed ISSUE-44490: Removed obsolete Authentication info in ajax request
* New Ajax object should be created instead of reusing old for CheckServerAvailability function to avoid ajax param changes

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-profile.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/ob-utilities.js
---

- Issue History
Date Modified Username Field Change
2020-06-25 14:20 shuehner New Issue
2020-06-25 14:20 shuehner Assigned To => Retail
2020-06-25 14:20 shuehner Resolution time => 1596146400
2020-06-25 14:20 shuehner Triggers an Emergency Pack => No
2020-08-17 07:22 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2020-08-17 09:16 ranjith_qualiantech_com Status new => scheduled
2020-08-18 03:51 hgbot Note Added: 0122107
2020-08-18 08:28 hgbot Resolution open => fixed
2020-08-18 08:28 hgbot Status scheduled => closed
2020-08-18 08:28 hgbot Note Added: 0122120
2020-08-18 08:28 hgbot Fixed in Version => RR20Q4
2020-08-18 08:28 hgbot Note Added: 0122121
2020-12-17 03:32 ranjith_qualiantech_com File Added: 44490_20Q1_mobilecore.diff


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker