Skip to content

providers/proxy: preserve URL-encoded path characters in redirect (cherry-pick #20476 to version-2026.2)#20482

Merged
BeryJu merged 1 commit intoversion-2026.2from
cherry-pick/20476-to-version-2026.2
Feb 23, 2026
Merged

providers/proxy: preserve URL-encoded path characters in redirect (cherry-pick #20476 to version-2026.2)#20482
BeryJu merged 1 commit intoversion-2026.2from
cherry-pick/20476-to-version-2026.2

Conversation

@authentik-automation
Copy link
Contributor

Cherry-pick of #20476 to version-2026.2 branch.

Original PR: #20476
Original Author: @Brolywood
Cherry-picked commit: 17ab3a4

…0476)

Use r.URL.EscapedPath() instead of r.URL.Path when building the
redirect URL in redirectToStart(). The decoded Path field converts
%2F to /, which url.JoinPath then collapses via path.Clean, stripping
encoded slashes from the URL. EscapedPath() preserves the original
encoding, fixing 301 redirects that break apps like RabbitMQ which
use %2F in their API paths.
@netlify
Copy link

netlify bot commented Feb 23, 2026

Deploy Preview for authentik-integrations ready!

Name Link
🔨 Latest commit 1f91e79
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/699c8110c95908000801c911
😎 Deploy Preview https://deploy-preview-20482--authentik-integrations.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Feb 23, 2026

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 1f91e79
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/699c8110c76d34000805ceed
😎 Deploy Preview https://deploy-preview-20482--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@BeryJu BeryJu merged commit cd75fe2 into version-2026.2 Feb 23, 2026
86 of 88 checks passed
@BeryJu BeryJu deleted the cherry-pick/20476-to-version-2026.2 branch February 23, 2026 17:10
@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.30%. Comparing base (ac7a4f8) to head (1f91e79).
⚠️ Report is 2 commits behind head on version-2026.2.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                Coverage Diff                 @@
##           version-2026.2   #20482      +/-   ##
==================================================
- Coverage           93.30%   93.30%   -0.01%     
==================================================
  Files                 978      978              
  Lines               54697    54697              
==================================================
- Hits                51036    51034       -2     
- Misses               3661     3663       +2     
Flag Coverage Δ
conformance 37.67% <ø> (+<0.01%) ⬆️
e2e 43.56% <ø> (+0.03%) ⬆️
integration 22.52% <ø> (+<0.01%) ⬆️
unit 91.50% <ø> (+<0.01%) ⬆️
unit-migrate 91.53% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants