Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for Forward Auth #1221

Open
1 task done
hiasr opened this issue Jan 4, 2025 · 3 comments · May be fixed by #580
Open
1 task done

Support for Forward Auth #1221

hiasr opened this issue Jan 4, 2025 · 3 comments · May be fixed by #580
Labels
awaiting triage This issue needs to be reviewed enhancement New feature or request

Comments

@hiasr
Copy link

hiasr commented Jan 4, 2025

Description

Currently Jellyseerr only supports password based authentication. While full OIDC support is in the works, as seen in the roadmap, it could be nice to support Forward Auth. Which would enable users to use their existing SSO with minimal effort.

Desired Behavior

When checking the authentication upon page load, check whether the request contains a certain header (i.e X-authentik-username), which should be predefined or configurable in settings. Jellyseerr can trust that the header contains the username of an authenticated user. The general flow of new user/first admin/... can follow the current flow of a new Jellyfin user.

Additional Context

I would be willing to work on this, but first want to validate whether this is a contribution you would be willing to accept!

Code of Conduct

  • I agree to follow Jellyseerr's Code of Conduct
@hiasr hiasr added awaiting triage This issue needs to be reviewed enhancement New feature or request labels Jan 4, 2025
@Fallenbagel
Copy link
Owner

#580

@hiasr
Copy link
Author

hiasr commented Jan 4, 2025

My bad, only searched in issues 😅

@hiasr hiasr closed this as completed Jan 4, 2025
@Fallenbagel
Copy link
Owner

My bad, only searched in issues 😅

No no you can keep this issue opened since there is none for it. This issue can be closed once the pr is merged

@Fallenbagel Fallenbagel reopened this Jan 4, 2025
@Fallenbagel Fallenbagel linked a pull request Jan 4, 2025 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting triage This issue needs to be reviewed enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants