Doorman needs to allow existing script files (cron.php and xmlrpc.php) to continue working without interference.

CommentFileSizeAuthor
#4 doorman_cron_passthrough.patch676 bytesemilcarpenter
#1 doorman-allowcron-1473370-1.patch842 bytesjrbeeman

Comments

jrbeeman’s picture

jrbeeman
he/him
Primary language English
Location Arizona, USA
commented
StatusFileSize
new doorman-allowcron-1473370-1.patch842 bytes

The attached patch addresses this issue. Due to the growing number of cases being checked for, I broke the existing checks out into a few lines where it made sense.

manarth’s picture

manarth commented
Status: Active » Fixed

Fixed in 7.x-1.x-dev (I'll give it a couple weeks or so and roll a 1.0 release if there's nothing major in the issue queue).
Nice spot, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

emilcarpenter’s picture

emilcarpenter commented
Issue summary: View changes
StatusFileSize
new doorman_cron_passthrough.patch676 bytes

I needed to patch version 1.1 for cron from outside to get through to Drupal. I did this on an old site, inherited from someone else, with some Features and without a database - so not tested on clean install.

emilcarpenter’s picture

emilcarpenter commented
Status: Closed (fixed) » Needs review

  • manarth committed 4527367 on 7.x-1.x authored by emilcarpenter 
    Issue #1473370 by emilcarpenter: Need to allow cron.php and xmlrpc.php...
manarth’s picture

manarth commented
Status: Needs review » Fixed

Thanks emilcarpenter, fixed in commit 4527367f160970f19444e929e7f5f192fd34fc5c.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.