Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
Hi, I just updated to the dev version (7.x-1.x-dev) to fix a problem with fast_404 not working with the D7 redirect module. Now (after the update) fast_404 does work with the redirect module but it does not work with drush.
On the command line I'm getting a 404 page returned for even just a "drush st" command.
Here is the output:
drush st
<html xmlns="http://www.w3.org/1999/xhtml"><head><title>404 Not Found</title></head><body><h1>Not Found</h1><p>The requested URL "/index.php" was not found on this server.</p></body></html>Drush command terminated abnormally due to an unrecoverable error.
I don't see any notes or documentation on how to get drush working in the inc files anywhere.
Thanks for any help.
Comment | File | Size | Author |
---|---|---|---|
#8 | fast_404-skip-cli-2114479-8.patch | 534 bytes | loopduplicate |
#7 | fast_404-skip-cli-2114479-7.patch | 536 bytes | loopduplicate |
#5 | fast_404-skip-cli-2114479-5.patch | 430 bytes | loopduplicate |
#3 | fast_404-ignore-drush-2114479-3.patch | 418 bytes | joelstein |
Comments
Comment #1
dshumaker CreditAttribution: dshumaker commentedTo get around the drush issue I just added a cli check in the settings.php that seemed to fix the problem for me.
So I guess this is a fixed bug but someone should probably document it. I'll let the maintainer close it.
Comment #2
loopduplicateThanks Dan! Perfect.
Comment #3
joelstein CreditAttribution: joelstein commentedHere's a patch which ignores drush calls in fast_404_ext_check(), instead of requiring users to update settings.php.
Comment #4
Dave ReidJust checking drupal_is_cli() is necessary. This shouldn't rely on Drush necessarily but should be bypassed alltogether if being invoked via command line.
Comment #5
loopduplicateUploading a new patch per Dave's recommendation.
Comment #6
adammaloneThis patch doesn't apply - was it created against the 7.x-1.x branch?
Comment #7
loopduplicateRerolling patch. This is definitely against the 7.x-1.x branch, not sure about the last one :)
Comment #8
loopduplicateRemoves whitespace.
Comment #10
adammalone.
Comment #11
adammaloneThanks!