-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Remove ACL Firewall before 4.0 release #14630
Comments
Implemented |
Branch with firewall https://github.com/phalcon/cphalcon/tree/4.0.x-with-firewall |
First of all thank you to all programmers and donors who make Phalcon possible. Happy holidays! That said, next time please release another BETA or RC version if documentation for some features is not complete. Because this v4.0.0 seems incomplete by removed ACL firewall. I'm sure all of us would rather wait a bit longer than have milestone version incomplete. Also, I think that basic access authentication like "HTTP Basic Auth" and "HTTP Digest Auth" should be a part of this ACL firewall. |
We want to deliver 4.0 fully documented. Currently we are struggling with the ACL firewall documentation. It's unclear how to exactly use it and how it internally works.
It's a really nice new feature for Phalcon but before we release it we must make sure we fully understand it so we can document it and help out community when they have questions.
For the release I want to rollback these changes and add them as a NFR for 4.1.
This is the main pull #14078
I see the following related pulls.
#12561
#11590
#14450
#14455
The text was updated successfully, but these errors were encountered: