- 🇮🇳India suresh prabhu parkala Bangalore
Re-rolled the patch to the latest 7.x. please review.
- last update
over 1 year ago Patch Failed to Apply - Status changed to Needs work
over 1 year ago 9:40am 14 June 2023 - 🇸🇰Slovakia poker10
Yes, because
system_update_7086
was introduced in Drupal 7.98.I am going to paste my comment from other issue also here: In case someone has run the update hook from this patch, it is likely that the real
system_update_7086
introduced by 📌 Improve security of session ID against DB exposure or SQL injection Fixed in Drupal 7.98 will not run. This will cause that these sites would not use the security improvement, because the session hashing will not be enabled. See: https://www.drupal.org/node/3364841 →I recommend that all sites using this patch should carefully check that change record and a committed patch and take measures to ensure that they have the session hashing enabled.
- 🇳🇱Netherlands cmseasy
Could someone create a patch for Drupal 7.98.
I am not a coder but would like to patch my site with the correct patch.
Thanks. - First commit to issue fork.
- last update
about 1 year ago run-tests.sh exception - @rpayanm opened merge request.
- Status changed to Needs review
about 1 year ago 5:52pm 16 August 2023 - last update
about 1 year ago 2,159 pass - last update
about 1 year ago 2,159 pass - last update
about 1 year ago run-tests.sh exception - last update
about 1 year ago 2,159 pass - 🇸🇰Slovakia poker10
It seems like it was only a random failure in the MR. I triggered the test again and it is green now.
- 🇩🇪Germany donquixote
I just saw this issue at the top of the list.
Reminds me of this old one from long ago, #512962: Optimize menu_router_build() / _menu_router_save() →
And now I see you already mention it.Adding the link.
- last update
8 months ago 2,179 pass - 🇳🇱Netherlands helmo
Here's an updated patch for Drupal 7.100.
It bumps the update hook number since core also added `system_update_7087`