You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think FuelPHP needs a separate security package, that will contain all classes and method related to security:
Crypt
Validation and cleaning
CSRF protection
PHPSecLib
There is a composer package for PHPSecLib. Biggest problem is that it's loaded with trigger_error()/user_error() calls instead of exceptions, which kinda sucks...
There is already some security code in the Foundation package, this will have to move too. This also means there won't be a need for a separate Crypt package.
The text was updated successfully, but these errors were encountered:
well tbh I think Security and FTP aren't kinda related. For me seems kinda out of scope and maybe better would be Networking package for FTP/SFTP etc? and that could depend on Security for some features.
I agree with @davispuh, the ftp package should have a dependency on the security package, or just the one lib. Nothing more. Would be odd to place it here.
But then again neither does SFTP. or SSH, or... I can live with a generic Networking package and a Security package, both requiring PHPSecLib, with the former exposing SFTP and SSH.
I think FuelPHP needs a separate security package, that will contain all classes and method related to security:
There is a composer package for PHPSecLib. Biggest problem is that it's loaded with trigger_error()/user_error() calls instead of exceptions, which kinda sucks...
There is already some security code in the Foundation package, this will have to move too. This also means there won't be a need for a separate Crypt package.
The text was updated successfully, but these errors were encountered: