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

Is this package still maintained? #369

Open
petrnymsa opened this issue Apr 5, 2022 · 6 comments
Open

Is this package still maintained? #369

petrnymsa opened this issue Apr 5, 2022 · 6 comments

Comments

@petrnymsa
Copy link

There is a lot of issues old more than one year without any responses. Pull Requests are the same.

Is this package maintained?

It would not bother me if the package's usage was for example for UI but this package aims to solve "security" within our app and I am not sure if is it ok to use the package which seems is a little bit abandoned.

@jfahrenkrug
Copy link

I wonder the same thing. Since it's under the dart-lang umbrella it seems like the best choice for OAuth in Dart. But maybe that's not the case anymore? Are there more mature packages to consider?

@petrnymsa
Copy link
Author

petrnymsa commented Apr 22, 2022

@jfahrenkrug Well just another one https://pub.dev/packages/oauth2_client, but i was not able to make it work. So far in our project we are using this package, but still really concerned about it's future and security.

It seems like in the Flutter world everyone just use Firebase Auth and they are good to go. It really suprise me, that there is no decent OAuth client

@RaphaelStAlme
Copy link
Contributor

Same opinion. That why i fork this project in order to resolve issue and develop oauth 2.0.1 version for my app. But yes it's a security risk if package is not maintained.

@petrnymsa
Copy link
Author

@RaphaelStAlme Could you share which fixes you did?

@RaphaelStAlme
Copy link
Contributor

RaphaelStAlme commented Apr 27, 2022

@RaphaelStAlme Could you share which fixes you did?

Yes of course @petrnymsa ,

I fixed issue with "expire in" type which is a string for Azure Directory, so i added a parseInt in order to have Int for all OAuth like Ad.

Before, it generated a typeException because of that

That it's 😉

@devoncarew
Copy link
Member

There is a lot of issues old more than one year without any responses. Pull Requests are the same.
Is this package maintained?

Good question! We are re-thinking whether we should own and publish this package now; I created an issue to track that: #375; comments welcome.

For background, this was created by the original pub team for use in the pub command-line tool. We likely don't have bandwidth to maintain it as a general purpose solution however. That's not ideal for two reasons; first, people assume that if a package is published from the Dart team, that it's intended to be a best-of-breed solution. And second, as this package is in the security space, having the package be well maintained becomes even more important.

@mosuem mosuem transferred this issue from dart-archive/oauth2 Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants