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

niteman’s picture

I have workarounded the issue adding "getenv('HTTP_HOST') ." prior to each call to session_name()

Sorry but i can't make the corresponding patch :(

darren oh’s picture

Status: Active » Postponed (maintainer needs more info)

We don't do anything with the files table. What other tables are you sharing?

darren oh’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)

I believe it is an error to use the same session name for multiple sites.

niteman’s picture

Component: Code » Documentation
Status: Closed (won't fix) » Active

Yes, 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.

darren oh’s picture

Status: Active » Postponed (maintainer needs more info)

I 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.

niteman’s picture

You 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.

...
/**
 * We try to set the correct cookie domain. If you are experiencing problems
 * try commenting out the code below or specifying the cookie domain by hand.
 */
/*
if (isset($_SERVER['HTTP_HOST'])) {
  $domain = '.'. preg_replace('`^www.`', '', $_SERVER['HTTP_HOST']);
  // Per RFC 2109, cookie domains must contain at least one dot other than the
  // first. For hosts such as 'localhost', we don't set a cookie domain.
  if (count(explode('.', $domain)) > 2) {
    ini_set('session.cookie_domain', $domain);
  }
}*/
ini_set('session.cookie_domain', 'yourdomain.com');
...

If you need more info/testing I'm at your disposal.

niteman’s picture

Status: Postponed (maintainer needs more info) » Active

If futher info is needed please follow up the issue telling me what you need.

darren oh’s picture

Component: Documentation » Code
Assigned: Unassigned » darren oh

No further information is needed. I'll have a fix out soon.

darren oh’s picture

Status: Active » Fixed

Fixed in CVS commit 87102.

hass’s picture

Status: Fixed » Patch (to be ported)

@Darren Oh: don't we need a backport for 1.x branch?

darren oh’s picture

Ported in CVS commit 88252, for those who want to stick witn the 1.x series. However, I recommend upgrading to XML Sitemap instead.

darren oh’s picture

Status: Patch (to be ported) » Fixed
darren oh’s picture

Version: 5.x-1.x-dev » 5.x-1.11
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.