feat: use the new logout endpoint (v3-apps)#4171
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Magnusrm
left a comment
There was a problem hiding this comment.
LGTM, the changes in percy seems to come from a comparison towards main. The failing cypress tests seem to come from the app "anonymous-stateless-app-v3" being unpublished
Description
Switches the "Logg ut"-link to the new Altinn 3 platform endpoint.
https://{altinnHost}/ui/authentication/LogOuthttps://platform.{altinnHost}/authentication/api/v1/logoutChanges
src/utils/urls/urlHelper.ts—logoutUrlAltinn(url)now usesextractHostFromUrlto build the platform host. Local/unknown origins return/(was/ui/authentication/LogOut/nullui/authentication/LogOut).src/utils/urls/urlHelper.test.ts— Tightened assertions from.toContainto.toBeand added cases for local/unknown origins.The single callsite (
AltinnAppHeaderMenu.tsx) is untouched.To be able to test this we do need to build locally and upload the build to CDN.
Related Issue(s)
Verification/QA
kind/*andbackport*label to this PR for proper release notes grouping