Skip to content

Draft: implement OAuth2#7

Draft
knuxify wants to merge 22 commits intodevelopfrom
authlib
Draft

Draft: implement OAuth2#7
knuxify wants to merge 22 commits intodevelopfrom
authlib

Conversation

@knuxify
Copy link
Copy Markdown
Member

@knuxify knuxify commented Aug 29, 2021

Very work-in-progress.

Current status: see todo list

Web settings code is still rough around the edges, and doesn't have any of the new features (these will probably be implemented when I'll redesign the settings). I'm planning a settings redesign in the near future, which will involve throwing out all templates and rewriting some web code once again (although this time I moved the rendering stuff to a generic function so adapting the code will be relatively quick). For the time being, I'll probably just stick with the existing templates.

The commits will eventually be squashed, this is absolutely not the final commit history.

  • Write basic tests for authentication
  • Fix web client editing
  • Clean up API tests in preparation for authentication
  • Set up username validation
  • Improve permission handling (https://twitter.com/knuxify/status/1445016302096109571) (out of scope for this MR, moving to to-do list)
  • Make API use authentication
  • Fix API tests to work with authentication
  • Add markup for OAuth pages? (might move this one to the "redesign" task)

Entirely untested (except for linting), breaks a lot of stuff. More
commits to implement authentication will follow.
@knuxify knuxify mentioned this pull request Aug 29, 2021
37 tasks
@knuxify knuxify added the help wanted Extra attention is needed label Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant