Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
For string $db_url values, the current code for fast_404_validate_path_mysql() only extracts the first character from $db_url and then tries to parse it:
This patch fixes things:
Comment | File | Size | Author |
---|---|---|---|
Selection_651.png | 72.43 KB | janusman | |
Selection_650.png | 65.51 KB | janusman | |
fast_404_validate_path_mysql-string-dburl-fix.patch | 1.01 KB | janusman | |
Comments
Comment #1
adammaloneAlejandro! I'm pushing for the patch in #2078863: Provide PDO db access in fast_404_path_check [patch attached] to be committed to use PDO instead of mysql functions. I'm hoping this will mitigate these issues. If you could test that I'd be happy to make that the defacto standard for this module (I wrote the patch prior to becoming a committer so don't feel it's right to merge that without a review!)
Comment #2
adammaloneRemoving comment duplicate.
Comment #3
adammaloneWait, I just saw this was for the 6.x version of the module.
Comment #5
adammaloneCommitted to 6.x thanks!
Comment #6
adammaloneCommitted to 6.x thanks!