Commit 6dbfc2f
chore(deps): upgrade itk-dev/openid-connect-bundle to 5.0
Bumps itk-dev/openid-connect-bundle (and upstream itk-dev/openid-connect)
from 4.x to 5.0. The 5.0 release reworks the exception hierarchy around a
marker interface: concrete exceptions no longer extend the now-deprecated
ItkOpenIdConnectException, so existing `catch (ItkOpenIdConnectException)`
blocks would silently match nothing.
Migrate the three OIDC catch sites in AuthOidcController and the one in
AzureOidcAuthenticator to catch OpenIdConnectExceptionInterface, the new
marker that both bundle- and library-thrown exceptions implement. Behaviour
(404 unknown provider, 500 OIDC failure, null end-session URL, wrapped auth
failure) is unchanged.
Add regression tests:
- AuthOidcControllerTest: the three getUrls() exception-mapping branches
(null end-session URL, 404, 500) that the upgrade put at risk.
- AzureOidcAuthenticatorTest: the previously untested authenticate() logic
— internal/external user provisioning, group-claim to tenant-role mapping,
multi-role accumulation, tenant de-provisioning, and the
unknown-suffix / missing-claim / unsupported-provider rejections.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>1 parent 89ba611 commit 6dbfc2f
7 files changed
Lines changed: 550 additions & 64 deletions
File tree
- src
- Controller/Api
- Security
- tests
- Controller/Api
- Security
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
7 | 12 | | |
8 | 13 | | |
9 | 14 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments