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

LDAP and SSO Support #7047

Open
1 task done
ashvinnihalani opened this issue Feb 19, 2022 · 6 comments
Open
1 task done

LDAP and SSO Support #7047

ashvinnihalani opened this issue Feb 19, 2022 · 6 comments
Labels

Comments

@ashvinnihalani
Copy link

ashvinnihalani commented Feb 19, 2022

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe

Lack of multiuser support

Describe the solution you'd like

Support for LDAP/SSO authentication. This provided many benefits:

  • Multiple user accounts without password sharing
  • Don't store admin credentials in plaintext config
  • Use same credentials/sessions across apps in this space ( i.e Radarr, NZBGet, Tautulli, etc)
  • Opens the door for user permissions (authorization) down the road
  • Improve security by offloading AuthN/AuthZ to services designed for these purposes

Describe alternatives you've considered

No real alternatives are other than using a different project. Keeping a proxy in front of the solution fails in certain use cases

Anything else?

There has been an issue opened in the past #2680 but it was closed due to lack of manpower. I am happy to help contribute/lead the development of this feature if it is not a high priority. .NET is not my strongest language so some pointers to where the relevant files to base off/refer to are located

AB#2609

@ashvinnihalani ashvinnihalani added Status: Needs Triage New Issue needing triage Type: Feature Request Issue is a feature request. labels Feb 19, 2022
@ta264
Copy link
Contributor

ta264 commented Feb 20, 2022

For the record, passwords are not stored in plaintext.

@bakerboy448 bakerboy448 added Status: Maybe One Day and removed Status: Needs Triage New Issue needing triage labels Feb 23, 2022
@aetaric
Copy link

aetaric commented Aug 28, 2023

With radarr 5.x requiring authentication (8911386), it might be time to revisit this. A non-zero number of people have authentication off in radarr since auth is offloaded to something else like apache2's mod_auth_mellon for SAML and authnz_ldap for LDAP.

@bakerboy448
Copy link
Contributor

A non-zero number of people have authentication off in radarr since auth is offloaded to something else like apache2's mod_auth_mellon for SAML and authnz_ldap for LDAP.

it's a non-issue and External auth is supported https://wiki.servarr.com/radarr/faq#forced-authentication

@RobinDadswell
Copy link
Member

With radarr 5.x requiring authentication (8911386), it might be time to revisit this. A non-zero number of people have authentication off in radarr since auth is offloaded to something else like apache2's mod_auth_mellon for SAML and authnz_ldap for LDAP.

We are currently investigating Plex auth and OIDC for extension of the current auth methods in the future, although there are some kinks to work out.

@Robin-Sch
Copy link

Is there any update for this?

@RobinDadswell
Copy link
Member

Is there any update for this?

Currently no update on this, have been having some issues with OIDC in the limited testing that's been done and trying to work through those kinks before we make this more widely available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants