Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036861Retail ModulesWeb POSpublic2017-09-14 19:092017-09-25 14:55
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q4Fixed in Version 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036861: Support authentication token also in case of single server environments
DescriptionThe mobile key authenticator [1] supports automatic relogin from webpos to the server in case the session gets lost on the server. This is a feature which should also work in case multi server is not enabled, so in case of single server.


[1]
http://wiki.openbravo.com/wiki/Retail:Configuration_Guide#Mobile_Server_Authentication [^]
Steps To ReproduceSet the authenticator class in Openbravo properties to use the MobileKeyAuthenticator:
authentication.class=org.openbravo.mobile.core.authenticate.MobileKeyAuthenticationManager

Start the server as single server
Login to webpos
stop the server
wait a while
start the server
--> webpos will ask to relogin
Proposed SolutionAlso send authentication tokens to the client so that it supports mobile key authentication == automatic login
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-09-14 19:09mtaalNew Issue
2017-09-14 19:09mtaalAssigned To => mtaal
2017-09-14 19:09mtaalTriggers an Emergency Pack => No
2017-09-14 19:10mtaalSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=15902#r15902
2017-09-20 09:52hgbotCheckin
2017-09-20 09:52hgbotNote Added: 0099168
2017-09-20 09:52hgbotStatusnew => resolved
2017-09-20 09:52hgbotResolutionopen => fixed
2017-09-20 09:52hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4bb7dbe495f97dc94a70688c63421b4f08dc9e8e [^]
2017-09-20 09:54mtaalReview Assigned To => AugustoMauch
2017-09-20 18:55rafademiguelIssue Monitored: rafademiguel
2017-09-25 14:53AugustoMauchNote Added: 0099560
2017-09-25 14:53AugustoMauchStatusresolved => closed
2017-09-25 14:55hgbotCheckin
2017-09-25 14:55hgbotNote Added: 0099561

Notes
(0099168)
hgbot   
2017-09-20 09:52   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 4bb7dbe495f97dc94a70688c63421b4f08dc9e8e
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Sep 20 09:33:24 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4bb7dbe495f97dc94a70688c63421b4f08dc9e8e [^]

Fixes issue 36861: Support authentication token also in case of single server environments
Use the token authentication also if the multi-server preference is not set

OBMOBCUtils: move the session check code after the login
MobileCoreLoginHandler: instead of checking for preference check if the mobile key authenticator is installed to decide if a token should be sent to the client
MobileKeyAuthenticationManager: add utility method, do not set session id so that the code which creates the dbsession can create it

---
M src/org/openbravo/mobile/core/authenticate/MobileKeyAuthenticationManager.java
M src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java
M src/org/openbravo/mobile/core/utils/OBMOBCUtils.java
---
(0099560)
AugustoMauch   
2017-09-25 14:53   
Code reviewed and verified
(0099561)
hgbot   
2017-09-25 14:55   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: d91377edd0ec336c17930bdfcf1f8d5d6140fe4c
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Mon Sep 25 14:36:20 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d91377edd0ec336c17930bdfcf1f8d5d6140fe4c [^]

Related with issue 36861: Removes commented out code

---
M src/org/openbravo/mobile/core/authenticate/MobileKeyAuthenticationManager.java
---