Skip to content

Auditors authentication landing on wrong org - #2463

Merged
adelowo merged 5 commits into
mainfrom
bug-ISS-2249
Jun 5, 2026
Merged

Auditors authentication landing on wrong org#2463
adelowo merged 5 commits into
mainfrom
bug-ISS-2249

Conversation

@adelowo

@adelowo adelowo commented Jun 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

core/fga/model/base/organization.fga

Lines 20 to 23 in f62bcc6

define can_view: [service, user with time_based_grant] or (member and access) or (admin and access) or can_edit or can_view from parent
# specific check to allow the view of the org but not all org owned data
define can_view_org: can_view or auditor

@github-actions github-actions Bot added bug Something isn't working httpserve labels Jun 5, 2026
@adelowo
adelowo marked this pull request as ready for review June 5, 2026 20:11
@adelowo
adelowo requested a review from a team as a code owner June 5, 2026 20:11
@adelowo
adelowo enabled auto-merge (squash) June 5, 2026 20:59
@adelowo
adelowo merged commit 7c0e63e into main Jun 5, 2026
17 checks passed
@adelowo
adelowo deleted the bug-ISS-2249 branch June 5, 2026 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@golanglemonade golanglemonade golanglemonade approved these changes
@matoszz matoszz Awaiting requested review from matoszz matoszz is a code owner automatically assigned from theopenlane/blacksmiths
@LemonAid711 LemonAid711 Awaiting requested review from LemonAid711 LemonAid711 is a code owner automatically assigned from theopenlane/blacksmiths

Assignees

No one assigned

Labels

bug Something isn't working cli common httpserve local-development

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adelowo @golanglemonade