Closed (fixed)
Project:
Google Sitemap
Version:
5.x-1.11
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
28 Oct 2007 at 19:31 UTC
Updated:
30 Nov 2007 at 19:51 UTC
I've downloaded Development snapshot 5.x-1.x-dev 2007-Oct-23 in order to fix the issue related to Internationalitation module.
I have a multi-site configuration sharing some tables and with Internationalitation module. The drupal files directory is shared between sites in order to share user pictures.
It seems that current implementation saves sitemaps without taking care of wich site it belongs to. I'm getting in all sites the same sitemap.
Sorry for my bad english.
Comments
Comment #1
niteman commentedI have workarounded the issue adding "getenv('HTTP_HOST') ." prior to each call to session_name()
Sorry but i can't make the corresponding patch :(
Comment #2
darren ohWe don't do anything with the files table. What other tables are you sharing?
Comment #3
darren ohI believe it is an error to use the same session name for multiple sites.
Comment #4
niteman commentedYes, you have proved to be rigth, but if needed by design I think it must be documented that a new sesion is needed for each time you are going to install the module on a multi-site scenario.
Thanks in advance.
Comment #5
darren ohI have not been able to make using the same session for multiple sites work. If this is something that does work, I can fix XML Sitemap. Otherwise, I think it's a general site configuration error that has nothing to do with XML Sitemap.
Comment #6
niteman commentedYou can just override the domain cookie in settings.php when sharing users and sessions table in a multisite scenario in order to get the users loged to all sites at the same time.
If you need more info/testing I'm at your disposal.
Comment #7
niteman commentedIf futher info is needed please follow up the issue telling me what you need.
Comment #8
darren ohNo further information is needed. I'll have a fix out soon.
Comment #9
darren ohFixed in CVS commit 87102.
Comment #10
hass commented@Darren Oh: don't we need a backport for 1.x branch?
Comment #11
darren ohPorted in CVS commit 88252, for those who want to stick witn the 1.x series. However, I recommend upgrading to XML Sitemap instead.
Comment #12
darren ohComment #13
darren ohComment #14
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.