Fix OIDC login crash when groups claim is null #22

ryan · 2026-04-05T10:12:49-04:00

ryan commented

2026-04-05 10:12:49 -04:00

Summary

Use claims.get("groups") or [] instead of claims.get("groups", []) so that an explicit null value from the OIDC provider is coerced to an empty list
Fixes ValueError: ldap_groups is non nullable field, but null was passed during OIDC callback

Test plan

Log in via OIDC with a provider that returns "groups": null in claims
Log in via OIDC with a provider that omits the groups claim entirely
Log in via OIDC with a provider that returns valid groups array

🤖 Generated with Claude Code

## Summary - Use `claims.get("groups") or []` instead of `claims.get("groups", [])` so that an explicit `null` value from the OIDC provider is coerced to an empty list - Fixes `ValueError: ldap_groups is non nullable field, but null was passed` during OIDC callback ## Test plan - [ ] Log in via OIDC with a provider that returns `"groups": null` in claims - [ ] Log in via OIDC with a provider that omits the `groups` claim entirely - [ ] Log in via OIDC with a provider that returns valid groups array 🤖 Generated with [Claude Code](https://claude.com/claude-code)

ryan added 1 commit 2026-04-05 10:12:49 -04:00

Fix OIDC login crash when groups claim is null a6c698c6bd

Use `claims.get("groups") or []` instead of `claims.get("groups", [])`
so that an explicit `null` value is coerced to an empty list, preventing
a ValueError on the non-nullable ldap_groups field.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

ryan merged commit fa0f68e3b4 into main

2026-04-05 10:12:55 -04:00

ryan referenced this issue from a commit

2026-04-05 10:12:56 -04:00

Merge pull request 'Fix OIDC login crash when groups claim is null' (#22) from fix/oidc-null-groups into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ryan/simbarag#22