[Radiant] sitemap.js prevents use of /admin in subdirectory (was Re: HTTP_X_FORWARDED_HOST breaks Apache proxying)

Sean Cribbs seancribbs at gmail.com
Sat Jun 2 10:53:18 CDT 2007


Earl,

Could you submit a patch for that to the Trac? 

Sean

Earl Chew wrote:
> Earl Chew <earl_chew at yahoo.com> wrote: 
> There is a patch for Apache to add ProxyAddXHeaders:
>
> http://mail-archives.apache.org/mod_mbox/httpd-dev/200702.mbox/browser
>
> Setting this option Off causes Apache to refrain from inserting the HTTP_X_* headers making the proxying action transparent.
> After applying this, I was stumped for a while when trying to use /admin to edit the site. Things appear to work properly except that clicking on the page expander (the + to see the child pages) would result in file not found /admin/ui/pages/children/...
>
> I finally found this in public/javascripts/sitemap.js where:
>
>     new Ajax.Updater(
>       row,
>       '/admin/ui/pages/children/' + id + '/' + level,
>
> assumes the script is in the toplevel directory. I couldn't think of any way of fixing this without modifying the file. My fix was to make this path relative:
>
>     new Ajax.Updater(
>        row,
>        '../admin/ui/pages/children/' + id + '/' + level,
>  
> I also see document.cookie saves the path=/admin, but I can't see this used anywhere.
>
>        
> ---------------------------------
> Building a website is a piece of cake. 
> Yahoo! Small Business gives you all the tools to get online.
> _______________________________________________
> Radiant mailing list
> Post:   Radiant at lists.radiantcms.org
> Search: http://radiantcms.org/mailing-list/search/
> Site:   http://lists.radiantcms.org/mailman/listinfo/radiant
>
>   




More information about the Radiant mailing list