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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0049303
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Modules] Reportingmajorhave not tried2022-05-12 08:412022-08-29 13:17
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Regression introduced in release
Summary

0049303: Max supported allowed organization in SSO limited to 50

DescriptionCurrently we pass the list of readable organizations from Openbravo to the reporting server as a url parameter. There are often restrictions in the number of organizations we can pass due to url length. Therefore we do not send more than 50. If there are more then no organization is send and the data of all organizations is visible for the user. A warning [1] is printed in the log in that case.

[1]
2022-05-08 18:56:30,518 [http-478] WARN org.openbravo.reporting.integration.utils.ReportingIntegrationUtils - User Belen DIAZ ECEIZA has more than 50 namely: 98 readable orgs, assuming he/she can see all orgs in reporting server
Steps To ReproduceHave a user with more than 50 allowed organizations assigned to him/her. Open a report in the backoffice.
Proposed SolutionJasper has a 2000 char limit on the user attributes. Sending more than 50-60 id's won't work. The solution is to now only send 50 org ids, but ensure that always the current organization is in the list of org ids. If the user needs access to all orgs this can be accomplished with an existing preference.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0140129)
hgbot (developer)
2022-08-15 23:10

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.integration/-/merge_requests/4 [^]
(0140136)
hgbot (developer)
2022-08-16 08:17

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.integration/-/merge_requests/4 [^]
(0140137)
hgbot (developer)
2022-08-16 08:17

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.integration [^]
Changeset: 994f02db0197587ad50c5396262a37620df2f7fa
Author: Martin Taal <martin.taal@openbravo.com>
Date: 15-08-2022 23:09:24
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.integration/-/commit/994f02db0197587ad50c5396262a37620df2f7fa [^]

Fixed ISSUE-49303: add current org when max num of org is reached, log debug

---
M src/org/openbravo/reporting/integration/utils/ReportingIntegrationUtils.java
---
(0140465)
mtaal (manager)
2022-08-29 13:17

MR solves it

- Issue History
Date Modified Username Field Change
2022-05-12 08:41 mtaal New Issue
2022-05-12 08:41 mtaal Assigned To => mtaal
2022-06-10 11:57 nicola_uva Issue Monitored: nicola_uva
2022-08-15 23:10 hgbot Note Added: 0140129
2022-08-16 08:08 mtaal Proposed Solution updated
2022-08-16 08:17 hgbot Note Added: 0140136
2022-08-16 08:17 hgbot Resolution open => fixed
2022-08-16 08:17 hgbot Status new => resolved
2022-08-16 08:17 hgbot Note Added: 0140137
2022-08-29 13:17 mtaal Note Added: 0140465
2022-08-29 13:17 mtaal Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker