Skip to content

Use configured backoffice route if available#16660

Merged
nikolajlauridsen merged 1 commit into
v14/devfrom
v14/bugfix/16179-use-configured-backoffice-route-if-available
Jun 27, 2024
Merged

Use configured backoffice route if available#16660
nikolajlauridsen merged 1 commit into
v14/devfrom
v14/bugfix/16179-use-configured-backoffice-route-if-available

Conversation

@bergmania

Copy link
Copy Markdown
Member

Description

Use the application url from configuration to initialise openiddict. If it is not available, it will fallback to the requested path (like before)

This fixes issues where umbraco is behind a proxy.

Fixes #16179

Test

  • Ensure login works with the configured application url
  • Ensure login works without a configured application url

…vailable and just fallback to the one from the first request.

Fixes #16179

@nikolajlauridsen nikolajlauridsen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, tests good 👍

@nikolajlauridsen nikolajlauridsen merged commit 1ea65c1 into v14/dev Jun 27, 2024
@nikolajlauridsen nikolajlauridsen deleted the v14/bugfix/16179-use-configured-backoffice-route-if-available branch June 27, 2024 08:47
nikolajlauridsen pushed a commit that referenced this pull request Jun 27, 2024
…vailable and just fallback to the one from the first request. (#16660)

Fixes #16179
matthewcare pushed a commit to matthewcare/Umbraco-CMS that referenced this pull request Jun 29, 2024
…vailable and just fallback to the one from the first request. (umbraco#16660)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants