openedx / openedx/openedx-proposals
Update OEP-42 Authentication proposing consolidation of authentication code
@robrap is already working on this.
Since Nov 17, 2023.
- 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.