Skip to content

OpenID Connect redirect URLs#383

Merged
rimi-itk merged 1 commit intodevelopfrom
feature/openid-connect-redirects
Mar 17, 2026
Merged

OpenID Connect redirect URLs#383
rimi-itk merged 1 commit intodevelopfrom
feature/openid-connect-redirects

Conversation

@rimi-itk
Copy link
Copy Markdown

@rimi-itk rimi-itk commented Mar 17, 2026

Link to ticket

https://leantime.itkdev.dk/_#/tickets/showTicket/6991

Description

Fixes OpenID Connect redirect URLs. I'm not sure what the defaults should actually be, but they should not contain http://os2loop.local.itkdev.dk.

phpfpm-1  | NOTICE: PHP message: Uncaught PHP Exception InvalidArgumentException: "The internal path component 'http://os2loop.local.itkdev.dk/user/logout' is external. You are not allowed to specify an external URL together with internal:/." at /app/web/core/lib/Drupal/Core/Url.php line 422

@rimi-itk rimi-itk changed the base branch from main to develop March 17, 2026 10:17
@rimi-itk rimi-itk force-pushed the feature/openid-connect-redirects branch from f5b19fb to d95d149 Compare March 17, 2026 10:25
@rimi-itk rimi-itk force-pushed the feature/openid-connect-redirects branch from d95d149 to e91774e Compare March 17, 2026 11:16
@rimi-itk rimi-itk marked this pull request as ready for review March 17, 2026 11:16
@rimi-itk rimi-itk requested a review from martinyde March 17, 2026 11:17
@rimi-itk rimi-itk merged commit 9f145da into develop Mar 17, 2026
15 checks passed
@rimi-itk rimi-itk deleted the feature/openid-connect-redirects branch March 17, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants