Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0052884POS2Corepublic2023-06-26 16:552023-06-29 09:40
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version23Q2.2Fixed in Version23Q2.2 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0052884: e.permissions is not iterable error when trying to log in
DescriptionUnder some circumstances, not clear yet, a "e.permissions is not iterable" error is displayed when trying to log in.

The problem seems to be caused by the application not being consistent when checking if a user has already logged in. Here [1] it checks if the session is active by checking if the logged user is not System, while here [2] it does it by checking if the user has a valid session. There are time where [2] evalues to true by [1] evaluates to false, resulting in the "e.permissions is not iterable" error.

To address the problem we will make sure the same condition (a combination of [1] and [2]) is checked in both places.

[1] https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/blob/master/src/org/openbravo/core2/login/Core2LoginUtilsServlet.java#L61 [^]
[2] https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/blob/master/src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java#L88 [^]
Steps To ReproduceNo steps to reproduce where found.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0052852 closed AugustoMauch e.permissions is not iterable error when trying to log in 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2023-06-29 09:31AugustoMauchTypedefect => backport
2023-06-29 09:31AugustoMauchTarget Version => 23Q2.2
2023-06-29 09:38hgbotNote Added: 0151804
2023-06-29 09:40hgbotNote Added: 0151805
2023-06-29 09:40hgbotNote Added: 0151806
2023-06-29 09:40hgbotResolutionopen => fixed
2023-06-29 09:40hgbotStatusscheduled => closed
2023-06-29 09:40hgbotFixed in Version => RR23Q2.2
2023-06-29 09:40hgbotNote Added: 0151807
2023-06-29 09:40hgbotNote Added: 0151808
2023-06-29 09:40hgbotFixed in VersionRR23Q2.2 => 23Q2.2
2023-06-29 09:40hgbotNote Added: 0151809

Notes
(0151804)
hgbot   
2023-06-29 09:38   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/560 [^]
(0151805)
hgbot   
2023-06-29 09:40   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1159 [^]
(0151806)
hgbot   
2023-06-29 09:40   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/560 [^]
(0151807)
hgbot   
2023-06-29 09:40   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: aab31a9f8bd2facae364b148a3c26ada6bd6a836
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 29-06-2023 09:38:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/aab31a9f8bd2facae364b148a3c26ada6bd6a836 [^]

Fixes ISSUE-52884: Be consistent when checking if the user is authenticatedd

Now both in MobileCoreLoginUtilsServlet and in Core2LoginUtilsServlet, the same condition will be check to
determine if the current user is authenticated. The isSessionActive has been moved from the latter to the
former

---
M src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java
---
(0151808)
hgbot   
2023-06-29 09:40   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1159 [^]
(0151809)
hgbot   
2023-06-29 09:40   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^]
Changeset: f3d47a71eae16ebd0bde8cf7244470cafb806df1
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 29-06-2023 09:39:17
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/f3d47a71eae16ebd0bde8cf7244470cafb806df1 [^]

Fixes ISSUE-52884: Be consistent when checking if the user is authenticated

Now both in MobileCoreLoginUtilsServlet and in Core2LoginUtilsServlet, the same condition will be check to
determine if the current user is authenticated. The isSessionActive has been moved from the latter to the
former

---
M src/org/openbravo/core2/login/Core2LoginUtilsServlet.java
---