-
Notifications
You must be signed in to change notification settings - Fork 450
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
Officially deprecate the package #698
Comments
I'm not a maintainer of this package, but I would be happy to take over maintenance and continue development of this package, even if it's just to ensure compatibility with the latest Symfony and PHP versions and to provide a migration path to thephpleague/oauth2-server-bundle |
I am not too into PHP or Packagist anymore these days.. Maybe @stof still is and IIRC he had access to that package on Packagist. Sorry. |
Thanks for the reply!
I figured, it was more because of perm. issues than anything, sorry I should have checked with stof first! |
Hi everyone, Thank you! |
Yes, you need to migrate.
Sadly, no one took the time to work on such a migration guide yet. One reason is that the path is not quite the same for every project: while the setup is pretty similar and straightforward, there're some differences that would be worth documenting at least. |
I think this package is pretty dead and there is a good replacement https://github.com/thephpleague/oauth2-server-bundle
Since this package is heavily used (over 5k daily installs still – without counting all the forks that just did a patch left and right to add support to a different doctrine or symfony version), I think an official statement and migration docs are in order.
My suggestion would be:
composer require league/oauth2-server-bundle
as a replacementThe first one is unfortunately an admin thing: no one can help but the maintainers themselves IIRC it's @deguif and @pierredup, so sorry for pinging you guys but no one else can do it.
Regarding the migration doc, it still can be a nice external contribution. I am sure a lot of people need to migrate and some might find time to document it. I am happy to help there for reviewing & merging doc PRs if you give me access to.
/cc @chalasr
The text was updated successfully, but these errors were encountered: