Piwigo / piwigo-sitemap

Creates a sitemap file for piwigo (plugin)
1 stars 1 forks source link

Make sitemap compatible with multisite #4

Open plegall opened 6 years ago

plegall commented 6 years ago

Hi @modus75

I wonder how it would be possible to make this plugin compatible with a multiple site install of Piwigo, like we have on Piwigo.com. Writing a sitemap.xml at the root of the Piwigo install obviously is a problem. Writing sitemap.xml outside root is also a problem, because you can't reference URLs outside the subdirectory of the sitemap.xml

The only solution I see would be to have a logical web server alias on sitemap.xml and physical file location would depend on nginx/apache configuration of each site.

modus75 commented 6 years ago

I don't see another solution than the one you propose. If apache/nginx pass a specific server variable containing a file system path, I could automatically save it there...

On Tue, Apr 3, 2018, 12:14 Pierrick Le Gall notifications@github.com wrote:

Hi @modus75 https://github.com/modus75

I wonder how it would be possible to make this plugin compatible with a multiple site install of Piwigo, like we have on Piwigo.com. Writing a sitemap.xml at the root of the Piwigo install obviously is a problem. Writing sitemap.xml outside root is also a problem, because you can't reference URLs outside the subdirectory of the sitemap.xml

The only solution I see would be to have a logical web server alias on sitemap.xml and physical file location would depend on nginx/apache configuration of each site.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/modus75/piwigo-sitemap/issues/4, or mute the thread https://github.com/notifications/unsubscribe-auth/AJRc9uXhgMDzHzBoLYRex0MJKZxSWOvaks5tk0tygaJpZM4TEzl3 .

plegall commented 6 years ago

OK, good idea. For example we could have a $_SERVER['PIWIGO_SITEMAP_LOCATION'] relative to Piwigo installation.

modus75 commented 6 years ago

It looks like the right choice

On Tue, Apr 3, 2018, 15:57 Pierrick Le Gall notifications@github.com wrote:

OK, good idea. For example we could have a $_SERVER['PIWIGO_SITEMAP_LOCATION'] relative to Piwigo installation.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/modus75/piwigo-sitemap/issues/4#issuecomment-378259152, or mute the thread https://github.com/notifications/unsubscribe-auth/AJRc9ulcT0nidvZifnSzZj0EGbIxSgj-ks5tk3_SgaJpZM4TEzl3 .