Skip to content

feat: add captive portal (zero-trust proxy) authentication support#695

Open
jerzykrlk wants to merge 1 commit into
BLeeEZ:masterfrom
jerzykrlk:feature/captive-portal
Open

feat: add captive portal (zero-trust proxy) authentication support#695
jerzykrlk wants to merge 1 commit into
BLeeEZ:masterfrom
jerzykrlk:feature/captive-portal

Conversation

@jerzykrlk
Copy link
Copy Markdown

Hello all,

This MR detects when the server is behind a zero-trust proxy (Cloudflare Access, Authelia, etc.) and present an embedded browser for authentication.

Cookies are persisted and forwarded to all request paths including audio streaming. Session is cleared on explicit logout but preserved on token expiry for seamless re-authentication.

It's slightly related to #694 - I just thought it would be useful for those who don't want to generate a private key.

Thanks!

Detect when the server is behind a zero-trust proxy (Cloudflare Access, Authelia, etc.) and present an embedded browser for authentication.

Cookies are persisted and forwarded to all request paths including audio streaming. Session is cleared on explicit logout but preserved on token expiry for seamless re-authentication.
@BLeeEZ
Copy link
Copy Markdown
Owner

BLeeEZ commented May 16, 2026

First of all thank you for providing PRs.
The problem I have is that I am not able to test this feature.
What would be needed to set up an environment to test this?

@jerzykrlk
Copy link
Copy Markdown
Author

Hello @BLeeEZ , thanks for the reply!

I didn't think of that before, apologies. I'll find something suitable and get back to you.

@rybomir
Copy link
Copy Markdown

rybomir commented May 27, 2026

@jerzykrlk I'd love to see this PR merged, so if there's anything I can do to help with testing (eg. I could set up a sample Navidrome instance behind a CF Zero Trust tunnel for @BLeeEZ) - please let me know!

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.

4 participants