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

Refactor ApiUserService to separate authentication vs authorization related methods #8049

Open
emyl3 opened this issue Aug 20, 2024 · 0 comments
Labels
Blocked Needs refinement Tech Debt issues dedicated to the resolution of technical debt

Comments

@emyl3
Copy link
Collaborator

emyl3 commented Aug 20, 2024

Background

This was created from a discussion on #8030's PR review.

ApiUserService is getting unwieldy. We should refactor this class so that authentication and authorization concerns are separated out.

Change requested

Refactor the ApiUserService class so that any methods related to authentication (resetting emails, passwords, etc...) are separate from authorization flows (getting orgRoles etc, which have been introduced thanks to the Okta migration work).

Acceptance criteria

  • ApiUserService is refactored so that
    • authentication flows are separate from authorization flows

Dependencies

Open questions

  • N/A

Notes

  • N/A

Additional context

  • N/A
@emyl3 emyl3 added Blocked Needs refinement Tech Debt issues dedicated to the resolution of technical debt labels Aug 20, 2024
@emyl3 emyl3 added this to the [Epic] Okta Migration milestone Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Needs refinement Tech Debt issues dedicated to the resolution of technical debt
Projects
None yet
Development

No branches or pull requests

1 participant