Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

UPDATE_PORTLET_LAYOUTS event is sent without user limitation #25556

Closed
erickgonzalez opened this issue Jul 19, 2023 · 0 comments · Fixed by #25683
Closed

UPDATE_PORTLET_LAYOUTS event is sent without user limitation #25556

erickgonzalez opened this issue Jul 19, 2023 · 0 comments · Fixed by #25683
Assignees
@erickgonzalez
Labels
OKR : Customer Success Owned by Arno OKR : Customer Support Owned by Scott Priority : 2 High QA : Not Needed Release : 23.01.5 Included in LTS patch release 23.01.5 Release : 23.08.16 Team : Bug Fixers Type : Defect

Comments

@erickgonzalez
Copy link
Contributor

erickgonzalez commented Jul 19, 2023 •
edited

Parent Issue

No response

Problem Statement

The event UPDATE_PORTLET_LAYOUTS is being sent without user limitation, so this causes the /menu endpoint gets called multiple times when using SAML to access the intranet. This makes the menu bar unusable.

Acceptance Criteria

Limit to the current user.

dotCMS Version

23.01+

Proposed Objective

Customer Support

Proposed Priority

Priority 2 - Important

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

https://dotcms.zendesk.com/agent/tickets/111821

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
@erickgonzalez erickgonzalez self-assigned this Jul 19, 2023
erickgonzalez added a commit that referenced this issue Jul 19, 2023
@erickgonzalez
possible fix for #25556 include in 23.01.5
1bf4bd7
@erickgonzalez erickgonzalez added Next LTS Release LTS: Excluded Ticket that has been excluded from at least one LTS and removed LTS: Excluded Ticket that has been excluded from at least one LTS labels Jul 21, 2023
erickgonzalez added a commit that referenced this issue Jul 26, 2023
@erickgonzalez
improvement #25556
60bd7e8
@erickgonzalez erickgonzalez added Release : 23.01.5 Included in LTS patch release 23.01.5 QA : Not Needed and removed Next LTS Release labels Aug 1, 2023
erickgonzalez added a commit that referenced this issue Aug 1, 2023
@erickgonzalez
#25556 throw event only for the user instead of global
79ebd80
@erickgonzalez erickgonzalez linked a pull request Aug 1, 2023 that will close this issue
#25556 throw event only for the user instead of global #25683
Merged
erickgonzalez added a commit that referenced this issue Aug 3, 2023
@erickgonzalez
#25556 throw event only for the user instead of global (#25683)
1a4d7e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erickgonzalez erickgonzalez

Labels
OKR : Customer Success Owned by Arno OKR : Customer Support Owned by Scott Priority : 2 High QA : Not Needed Release : 23.01.5 Included in LTS patch release 23.01.5 Release : 23.08.16 Team : Bug Fixers Type : Defect
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#25556 throw event only for the user instead of global dotCMS/core
1 participant
@erickgonzalez