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
AFAIK starting from PHP 8.0 the main security issue of assert() function implementation is removed. Thus given the minimum PHP requirement version for the Coding Standards is 8.1 the function should be removed from the list of the forbidden functions.
Expected behavior
No error The use of function assert() is forbidden is shown when checking the standards on Magento code.
Benefits
The function usage is allowed which provides reducing of boilerplate in code base.
Additional information
I'm not a security expert, so I might be wrong on the topic.
The text was updated successfully, but these errors were encountered:
Hi @oleksii-lisovyi. Thank you for your report.
To speed up processing of this issue, make sure that you provided sufficient information.
Add a comment to assign the issue: @magento I am working on this
Description
AFAIK starting from PHP 8.0 the main security issue of
assert()
function implementation is removed. Thus given the minimum PHP requirement version for the Coding Standards is 8.1 the function should be removed from the list of the forbidden functions.Expected behavior
No error
The use of function assert() is forbidden
is shown when checking the standards on Magento code.Benefits
The function usage is allowed which provides reducing of boilerplate in code base.
Additional information
I'm not a security expert, so I might be wrong on the topic.
The text was updated successfully, but these errors were encountered: