Openbravo Issue Tracking System - Retail Modules
View Issue Details
0034083Retail ModulesStoreServerpublic2016-07-22 14:222016-09-26 11:59
mtaal 
mtaal 
normalminorhave not tried
closedfixed 
5
 
RR16Q3.2RR16Q3.2 
AugustoMauch
No
0034083: Store server transitioning check does not use admin mode
When a user tries to run multi-server we see a low level error in the log:

org.openbravo.base.exception.OBSecurityException: Entity OBMOBC_SERVER_DEFINITION is not readable by the user B8D524AE26F245A0B8FA4407CCB78ED7
        at org.openbravo.dal.security.EntityAccessChecker.checkReadable(EntityAccessChecker.java:448)
        at org.openbravo.dal.service.OBDal.checkReadAccess(OBDal.java:601)
        at org.openbravo.dal.service.OBDal.checkReadAccess(OBDal.java:588)
        at org.openbravo.dal.service.OBDal.get(OBDal.java:297)
        at org.openbravo.mobile.core.servercontroller.MobileServerController.getThisServerDefinition(MobileServerController.java:311)
        at org.openbravo.mobile.core.servercontroller.MobileServerController.serverHasTransitioningStatus(MobileServerController.java:283)
        at org.openbravo.mobile.core.process.JSONProcessSimple.exec(JSONProcessSimple.java:52)
        at org.openbravo.mobile.core.process.SecuredJSONProcess.secureExec(SecuredJSONProcess.java:53)
        at org.openbravo.mobile.core.process.MobileServiceProcessor.execProcess(MobileServiceProcessor.java:117)
        at org.openbravo.mobile.core.process.MobileServiceProcessor.execServiceName(MobileServiceProcessor.java:79)
        at org.openbravo.mobile.core.process.MobileService.doGetOrPost(MobileService.java:89)
        at org.openbravo.mobile.core.process.MobileService.doPost(MobileService.java:59)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:646)
        at org.openbravo.base.HttpBaseServlet.serviceInitialized(HttpBaseServlet.java:228)
        at org.openbravo.base.secureApp.HttpSecureAppServlet.service(HttpSecureAppServlet.java:431)
        at org.openbravo.client.kernel.BaseKernelServlet.callServiceInSuper(BaseKernelServlet.java:87)
        at org.openbravo.mobile.core.process.WebServiceAuthenticatedServlet.service(WebServiceAuthenticatedServlet.java:60)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:727)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:303)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
See above.
Set admin mode in MobileServerController around the logic to read the server definition.
No tags attached.
blocks defect 0033530RR16Q4 closed mtaal Store server transitioning check does not use admin mode 
Issue History
2016-09-26 09:26mtaalTypedefect => backport
2016-09-26 09:26mtaalTarget VersionRR16Q4 => RR16Q3.2
2016-09-26 11:48hgbotCheckin
2016-09-26 11:48hgbotNote Added: 0090251
2016-09-26 11:48hgbotStatusscheduled => resolved
2016-09-26 11:48hgbotResolutionopen => fixed
2016-09-26 11:48hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/ad8dc963563fbf8cd77cab09e832fd61b0019be3 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q3.2/org.openbravo.mobile.core/rev/85a3a9d73264be811f914cb2a71df69c899cd603 [^]
2016-09-26 11:59AugustoMauchNote Added: 0090253
2016-09-26 11:59AugustoMauchStatusresolved => closed
2016-09-26 11:59AugustoMauchFixed in Version => RR16Q3.2

Notes
(0090251)
hgbot   
2016-09-26 11:48   
Repository: retail/backports/3.0RR16Q3.2/org.openbravo.mobile.core
Changeset: 85a3a9d73264be811f914cb2a71df69c899cd603
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Sep 26 11:48:13 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.2/org.openbravo.mobile.core/rev/85a3a9d73264be811f914cb2a71df69c899cd603 [^]

Fixes issue 34083: Store server transitioning check does not use admin mode
Set admin mode around get of the mobile server definition

---
M src/org/openbravo/mobile/core/servercontroller/MobileServerController.java
---
(0090253)
AugustoMauch   
2016-09-26 11:59   
Code reviewed and verified