Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0052884 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [POS2] Core | major | have not tried | 2023-06-26 16:55 | 2023-06-29 09:40 | |||
| Reporter | AugustoMauch | View Status | public | |||||
| Assigned To | AugustoMauch | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 23Q2.2 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | 23Q2.2 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0052884: e.permissions is not iterable error when trying to log in | |||||||
| Description | Under 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 Reproduce | No steps to reproduce where found. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0151804) hgbot (developer) 2023-06-29 09:38 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/560 [^] |
|
(0151805) hgbot (developer) 2023-06-29 09:40 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1159 [^] |
|
(0151806) hgbot (developer) 2023-06-29 09:40 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/560 [^] |
|
(0151807) hgbot (developer) 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 (developer) 2023-06-29 09:40 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1159 [^] |
|
(0151809) hgbot (developer) 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 --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2023-06-29 09:31 | AugustoMauch | Type | defect => backport |
| 2023-06-29 09:31 | AugustoMauch | Target Version | => 23Q2.2 |
| 2023-06-29 09:38 | hgbot | Note Added: 0151804 | |
| 2023-06-29 09:40 | hgbot | Note Added: 0151805 | |
| 2023-06-29 09:40 | hgbot | Note Added: 0151806 | |
| 2023-06-29 09:40 | hgbot | Resolution | open => fixed |
| 2023-06-29 09:40 | hgbot | Status | scheduled => closed |
| 2023-06-29 09:40 | hgbot | Fixed in Version | => RR23Q2.2 |
| 2023-06-29 09:40 | hgbot | Note Added: 0151807 | |
| 2023-06-29 09:40 | hgbot | Note Added: 0151808 | |
| 2023-06-29 09:40 | hgbot | Fixed in Version | RR23Q2.2 => 23Q2.2 |
| 2023-06-29 09:40 | hgbot | Note Added: 0151809 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |