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

(api) Allow multiple authentication methods (oidc & basic) [after #463] #449

Closed
wants to merge 4 commits into from

Conversation

Leobouloc
Copy link
Contributor

@Leobouloc Leobouloc commented Oct 5, 2023

WIP / Merge after feature/enforce-scopes (after #463 )

Purpose

Currently, an instance of Ralph LRS has a fixed auth method (basic auth and OIDC). The suggestion is to allow multiple auth methods. The use case could be: LMS write to Ralph LRS via basic Auth; while data analysts fetch from Ralph using OIDC.

Proposal

It is proposed to change RALPH_RUNSERVER_AUTH_BACKEND to RALPH_RUNSERVER_AUTH_BACKENDS and to make it accept comma separated lists as values RALPH_RUNSERVER_AUTH_BACKEND=basic,oidc.

The auth method for incoming requests will be chosen based on header information (basic, oidc)

  • implement suggestion
  • test that auth methods work individually and simultaneously

@Leobouloc Leobouloc changed the base branch from master to feature/enforce-scopes October 5, 2023 07:06
@Leobouloc Leobouloc added the WIP label Oct 5, 2023
@Leobouloc Leobouloc changed the title (api) Allow multiple authentication methods (api) Allow multiple authentication methods (oidc & basic) Oct 5, 2023
@Leobouloc Leobouloc changed the title (api) Allow multiple authentication methods (oidc & basic) (api) Allow multiple authentication methods (oidc & basic) [after #441] Oct 5, 2023
@Leobouloc Leobouloc changed the title (api) Allow multiple authentication methods (oidc & basic) [after #441] (api) Allow multiple authentication methods (oidc & basic) [after #463] Oct 17, 2023
@Leobouloc
Copy link
Contributor Author

Replaced by #473 for convenience.

@Leobouloc Leobouloc closed this Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant