gada121982 commented on PR #10437: URL: https://github.com/apache/gravitino/pull/10437#issuecomment-4065272136
@LauraXia123 Thanks for flagging! All Copilot comments have been addressed: 1. **oidc.test.js** — Updated tests to assert `access_token` preference (matching the new token selection logic) 2. **authn.yaml schema ref** — Moved from `#/components/responses/` to `#/components/schemas/` (proper OpenAPI structure) 3. **signout order** — `signoutRedirect()` now called before `clearAuthData()` so `id_token_hint` is available for RP-initiated logout 4. **PR description** — Updated all references from `/api/auth/me` to `/api/authn/me` 5. **Comment typo** — Fixed fallback comment to reference `/api/authn/me` All tests pass (Java unit tests + OIDC vitest + OpenAPI lint). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
