On 10/01/2009 05:08 PM, Cristian Ciupitu wrote:
I have a fix for it at http://github.com/ciupicri/cobbler/commit/507b38453f7cafe42f2e37b406b0856c7d... In case there's a git problem my repo is git://github.com/ciupicri/cobbler.git (the release20 branch, although main has it too) and the patch is also available in the bug tracker.
I've tested the patch and it works fine, except for a tiny problem. /etc/httpd/conf.d/cobbler.conf contains this line:
AliasMatch ^/cobbler(.*)?$ "/var/www/cobbler$1"
which means that the cobbler_webui_sessions directory will be shared just like cobbler_webui_content. I propose to replace the regular expression with an explicit list of directories. As "The Zen of Python" says it, "Explicit is better than implicit". Another option could be to rename cobbler_ui_sessions to something else, e.g. sessions_cobbler. I'm waiting for your suggestions on this, so I can send another patch.
If you're for the first solution, are there any other directories starting with cobbler besides these?
- cobbler
- cobbler_webui_content
- cobbler_api
- cobbler_web
- cobbler_track.... (which is covered by another AliasMatch line)
Thank you, Cristian
cobbler-devel mailing list cobbler-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/cobbler-devel
/var/www is really intended for user content, not even so much app content (which we need to be moving more of cobbler's support content to /usr/share over time -- data that we mirror lives nicely in /var/www and there is resistance to moving it, even with a symlink). Sharing sessions is obviously something we don't want to do, and we don't want someone to have to worry about that when configuring Apache on a shared server.
Sessions do belong in /var of course, but not /var/www.
A good location to me seems to be /var/lib/cobbler/web_sessions. We already own /var/lib/cobbler, and can put what we want there.
--Michael