Skip to content

fix(rooms): let portal schedule rooms under non-project groups - #50

Merged
zyx1121 merged 2 commits into
mainfrom
fix/service-group-check
Nov 6, 2026
Merged

zyx1121 merged 2 commits into
mainfrom
fix/service-group-check

Conversation

@zyx1121

@zyx1121 zyx1121 commented Nov 6, 2026

Copy link
Copy Markdown
Member

Why

portal books the Monday lab meeting under the GitLab group lab-meeting. It is not a Keycloak project group, so schedule_room threw 沒有這個專案群組. The 2026-11-09 lab meeting has no room (portal request room-booking-aa536b347684, failed 11-02). The 11-02 meeting was booked before the check existed.

Change

checkGroup skips the check when actor.service is set (a caller with a SERVICE_KEYS key, i.e. portal). Members, through the web or MCP, are still limited to project groups.

Verification

typecheck, lint, format and actions checks pass on the sandbox. After deploy, portal retries the 11-09 booking and gets a room.

🤖 Generated with Claude Code

zyx1121 and others added 2 commits November 6, 2026 13:27
The Monday lab meeting books its room under the GitLab group lab-meeting,
which is not a Keycloak project group, so schedule_room refused it with
沒有這個專案群組 and the 11-09 lab meeting has no room. A trusted service
caller now skips the project-group check; members are still checked.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zyx1121
zyx1121 merged commit af6f75c into main Nov 6, 2026
4 checks passed
@zyx1121
zyx1121 deleted the fix/service-group-check branch November 6, 2026 05:31
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.

1 participant