openedx / openedx/openedx-proposals

Update OEP-42 Authentication proposing consolidation of authentication code

Open
#541 9 comments 0 reactions 1 assignee View on GitHub

@robrap is already working on this.

Since Nov 17, 2023.

feb25 waiting on author
Dominant language
Python
Stars
50
Forks
47
Avg merge
1d 14h
Merged PRs (30d)
2

Description

Our authentication code is mostly consolidated under edx-drf-extensions and edx-platform's oauth_dispatch (as examples).

Some services use custom authentication code, such as edx-platform XBlocks code or notes custom JWT decoder. These special cases are usually discovered through breaking changes in Production.

This ticket is about codifying our wish to both consolidate authentication code in fewer locations, and to not introduce custom authentication code unless absolutely necessary. Even if the custom code is used in a single service, it is probably better to add it to the consolidated code base, because it almost certainly interacts with the rest of the authentication code.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.