splash preventing cron.php from working properly

guaka - June 10, 2009 - 16:41
Project:Splash
Version:6.x-2.4
Component:Documentation
Category:support request
Priority:normal
Assigned:Unassigned
Status:active
Issue tags:cron
Description

splash is preventing cron.php from working properly. Instead of executing cron.php the splash page is returned.

#1

guaka - June 10, 2009 - 16:54

I'm not sure if it's the Drupal way, but here's a fix:

59c59
<
---
>
75c75,78
<
---
>
> } elseif ($_SERVER['REQUEST_URI'] == '/cron.php') {
> $splash = FALSE;
>

#2

seanr - June 10, 2009 - 22:13

Before I commit this, I'd like to get a better understanding of what's going on. It doesn't seem like Drupal should be returning the splash page at all for cron.php. Exactly what settings do you have on each of the tabs?

#3

alyawn - June 16, 2009 - 03:07

A work around here is to call cron with ?splash=off and it works fine:

0 * * * * wget -O - -q -t 1 http://examplesite.com/cron.php?splash=off

#4

seanr - June 17, 2009 - 21:16
Component:Code» Documentation
Category:bug report» support request

Ah! Good call, thanks! I guess we'll need to add that to the documentation.

#5

guaka - June 18, 2009 - 17:53

It wasn't working properly and my patch fixed it. I didn't set up splash on the site where cron wasn't working anymore.

#6

guaka - June 18, 2009 - 17:56

cron calls drupal (and splash) with wget, and so the splash page gets returned (as it has never run on the wget 'browser' before), the actual cron page is working fine

Here's what we have in crontab: 35 * * * * www-data /usr/bin/wget -O - -q http://example.com/cron.php
When I tried it from the command-line wget was just fetching the splash page.

Thanks for the great module.

#7

seanr - June 22, 2009 - 20:23

guaka, if you undo your patch and change your cron to this, it should work just fine:

35 * * * * www-data /usr/bin/wget -O - -q http://examplesite.com/cron.php?splash=off

 
 

Drupal is a registered trademark of Dries Buytaert.