Skip to content

Remove Guest as an assignable role#437

Merged
andreasfrisch merged 5 commits into
developfrom
fix/guest-permission-and-role-removal
May 12, 2026
Merged

Remove Guest as an assignable role#437
andreasfrisch merged 5 commits into
developfrom
fix/guest-permission-and-role-removal

Conversation

@andreasfrisch
Copy link
Copy Markdown
Contributor

@andreasfrisch andreasfrisch commented May 8, 2026

🌟 Changes:

  • Filter out 'Guest' role from list of assignable roles (we still want the Guest role in the system, it is just not assignable).
  • Fail requests asking to assign the Guest role (should people attempt to use the API directly).
  • When checking permissions, always include the guest permissions (cached at the request level)

@andreasfrisch andreasfrisch requested a review from SEQUOIIA May 8, 2026 11:20
@DFDS-Snyk
Copy link
Copy Markdown

DFDS-Snyk commented May 8, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@andreasfrisch andreasfrisch merged commit 6840f58 into develop May 12, 2026
9 checks passed
@andreasfrisch andreasfrisch deleted the fix/guest-permission-and-role-removal branch May 12, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants