-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Added php8 to list of forbidden executable extensions #35000
Conversation
As it's for staging I've removed the "[3.10]" from the title. |
I have tested this item ✅ successfully on 595e238 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
I have tested this item ✅ successfully on 595e238 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
@SniperSister It's not sufficient to set the RTC label on GitHub to get the right status shown in the issue tracker. It needs to use the tracker and change status there. I've done that here now. |
Drone failing for unrelated reasons ... restarted to see if it succeeds this time. |
I have not tested this item. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
Back to pending. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
I have tested this item ✅ successfully on 595e238 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
I have tested this item 🔴 unsuccessfully on 595e238 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
Back to pending. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35000. |
Merging to sync up with 4.0: #35001 Thanks! |
Summary of Changes
Added PHP8 to the hardcoded list of executables
Testing Instructions
Verify by code review