Skip to content

feat: use the new logout endpoint (v3-apps)#4171

Open
framitdavid wants to merge 1 commit intov3from
feat/use-new-logout-endpoint-for-a3-in-v3-apps
Open

feat: use the new logout endpoint (v3-apps)#4171
framitdavid wants to merge 1 commit intov3from
feat/use-new-logout-endpoint-for-a3-in-v3-apps

Conversation

@framitdavid
Copy link
Copy Markdown
Contributor

@framitdavid framitdavid commented May 5, 2026

Description

Switches the "Logg ut"-link to the new Altinn 3 platform endpoint.

  • Before: https://{altinnHost}/ui/authentication/LogOut
  • After: https://platform.{altinnHost}/authentication/api/v1/logout

Changes

  • src/utils/urls/urlHelper.tslogoutUrlAltinn(url) now uses extractHostFromUrl to build the platform host. Local/unknown origins return / (was /ui/authentication/LogOut / nullui/authentication/LogOut).
  • src/utils/urls/urlHelper.test.ts — Tightened assertions from .toContain to .toBe and 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

  • Manual functionality testing
    • I have tested these changes manually
    • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
    • No testing done/necessary
  • Automated tests
    • Unit test(s) have been added/updated
    • Cypress E2E test(s) have been added/updated
    • No automatic tests are needed here (no functional changes/additions)
    • I want someone to help me make some tests
  • UU/WCAG (follow these guidelines until we have our own)
    • I have tested with a screen reader/keyboard navigation/automated wcag validator
    • No testing done/necessary (no DOM/visual changes)
    • I want someone to help me perform accessibility testing
  • User documentation @ altinn-studio-docs
    • Has been added/updated
    • No functionality has been changed/added, so no documentation is needed
    • I will do that later/have created an issue
  • Support in Altinn Studio
    • Issue(s) created for support in Studio
    • This change/feature does not require any changes to Altinn Studio
  • Sprint board
    • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
    • I don't have permissions to do that, please help me out
  • Labels
    • I have added a kind/* and backport* label to this PR for proper release notes grouping
    • I don't have permissions to add labels, please help me out

@framitdavid framitdavid added the kind/other Pull requests containing chores/repo structure/other changes label May 5, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 5, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b8f43685-5f24-4a4c-afcd-ce456d6c41c4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/use-new-logout-endpoint-for-a3-in-v3-apps

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@framitdavid framitdavid added the squad/utforming Issues that belongs to the named squad. label May 5, 2026
@framitdavid framitdavid moved this to 👷 In progress in Team Altinn Studio May 5, 2026
@framitdavid framitdavid moved this from 👷 In progress to 🔎 In review in Team Altinn Studio May 5, 2026
Copy link
Copy Markdown
Contributor

@Magnusrm Magnusrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/other Pull requests containing chores/repo structure/other changes squad/utforming Issues that belongs to the named squad.

Projects

Status: 🔎 In review

Development

Successfully merging this pull request may close these issues.

2 participants