Skip to content

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

Draft
framitdavid wants to merge 3 commits into
v3from
feat/use-new-logout-endpoint-for-a3-in-v3-apps
Draft

feat: use the new logout endpoint (v3-apps)#4171
framitdavid wants to merge 3 commits into
v3from
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: 44c21e27-0278-4363-9cee-0254890a3b7d

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

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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

@sonarqubecloud
Copy link
Copy Markdown

@framitdavid framitdavid marked this pull request as draft May 13, 2026 09:23
@framitdavid
Copy link
Copy Markdown
Contributor Author

Denne er blokkert til Team autorisasjon får sett på hvorfor vi ender opp i en evig loop mellom A2 og A3 redirection. Vi har en pågående tråd på intern Slack.

@framitdavid framitdavid moved this from 🔎 In review to ⚠️ Blocked in Team Altinn Studio May 13, 2026
@framitdavid framitdavid self-assigned this May 13, 2026
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: ⚠️ Blocked

Development

Successfully merging this pull request may close these issues.

3 participants