includes/session_keepalive.php

Hi,

I have an install of 0.9.5 beta 5 (rev 1805) - not sure if this is related to the beta or present in 0.9.2.1 as well.

Firstly, you need to know:
My modX install is at http://dev.andyshellam.eu/_alpha/ase_v62.
The directory http://dev.andyshellam.eu/_alpha is password-protected by .htaccess, so only I have access to it, so this can’t be from a search engine or spider, or it’d be a 403 - plus dev.andyshellam.eu was only created in the last week.

I’ve installed AWStats to monitor the statistics of the site for when it goes live, and have noticed a huge set of 404 errors (practically 1 per page hit.)

The 404 is for a page called /_alpha/includes/session_keepalive.php, referred by http://dev.andyshellam.eu/_alpha/ase_v62/manager/
Now, /_alpha/ase_v62/manager/includes/session_keepalive.php is the actual location of the file, but modX is skipping down 2 directories to find it, when it should be in the same directory.

I’ve had a quick look through most of the files included from manager/index.php but cannot find a reference to session_keepalive.php.

If this is a bug I’ll log it on the tracker but I’m not sure if it is.

Has anyone else noticed this in the same or a previous install?

It’s called from manager/media/script/session.js... It’s intended to keep the manager session open so that you don’t loose edits when editing long documents. Perhaps the path needs to be revised... Jeff?

Cheers Ryan,

Do you want me to log it on FS?

It likely should be ... and preferably accompanied with code that fixes it properly. Please!

I’m on it

Done. Have attached a replacement script.js in FS#619.

Here’s the results (from Apache log file):

Before edit (404 code + incorrect URL request):



After edit (302 code + correct URL request):

Thanks Andy!

Fix committed to my personal branch in SVN as commit #1829, it will need merging with the main trunk for testing.