New major
We want to make a new major soon were we would then remove the deprecations we added:
- IdentifierCollection (https://github.com/cakephp/authentication/pull/712)
I would also like to see the following fixed up:
- [x] CakeRouterUrl urlChecker (maybe called "Cake") auto defaults if we detect CakePHP context instead of "Default" ( https://github.com/cakephp/authentication/pull/714/commits/5607c7b70d86d5ff0ee25e3722591f54e73a9ebd ), or "Array vs String"
- [x] single URL by default, most apps only ever need a single app The array|string magic is not needed anymore then It internally accepts both string or array style as always in Cake
- [x] A MultiUrl checker can be added to specifically support multiple loginUrls
- [x] Constants for username/password shoud be at their respective classes instead of abstract, there are several ones that need either different naming or none of those pairs.
In general the next major could be more cake context bound and have it as a real dependency. makes things much easier and the plugin most likely has never been used in an external context.
In general the next major could be more cake context bound and have it as a real dependency.
I am fine with that. I highly doubt this plugin/lib is used independently without CakePHP, so we can just drop any Cake agnostic stuff.
Everything I found "officially" dependent on the auth plugin on packagist is definitely cakephp related, so making the auth plugins framework agnostic is not needed. https://packagist.org/packages/cakephp/authentication/dependents?order_by=downloads
Packagist won't show private project usage but again I highly doubt there are any.
Sure, but the only place where we indireclty say it is "framework agnostic" is the github description with
Authentication plugin for CakePHP. Can also be used in PSR7 based applications.
So I'd say the few users who are acutally using this outside of the cakephp framework will be able to find another solution.
I adjusted the major branch as per your feedback:
- Default (old Cake)
- Generic (old Default)
Hardcoded default, no more auto detect
Adjusted the migration guide accordingly.
@ADmad Are we ready for the release?