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

Simplify OAuth Handler #148

Closed
Barbapapazes opened this issue Aug 26, 2024 · 2 comments
Closed

Simplify OAuth Handler #148

Barbapapazes opened this issue Aug 26, 2024 · 2 comments

Comments

@Barbapapazes
Copy link
Contributor

Barbapapazes commented Aug 26, 2024

Hello,

There is a lot of code redundancy within OAuthHandler and I think there is a lot of possible refactor to simplify the code and to help maintenance.

I will make multiple PR to try to resolve this:

@atinux
Copy link
Owner

atinux commented Sep 2, 2024

Shall we close?

@Barbapapazes
Copy link
Contributor Author

I'll see.

But if I remember well, there is still a possible simplification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@atinux @Barbapapazes and others