Latest MapServer patch breaks GeoMoose

[kalxas]

For rc3 a mapserver patch was introduced:

​https://github.com/OSGeo/OSGeoLive/commit/5ddf0302b29a4868d7352cfed114ad46e84ea4b1

This patch is breaking GeoMoose (see attachment)

[sdlime]

Need to generalize the value for MS_MAP_PATTERN in Apache. What's the path to all the GeoMoose mapfiles?

[kalxas]

Files are available in various sub-folders of /usr/local/geomoose/gm3-demo-data/

[jimk]

/usr/local/geomoose/gm3-demo-data/

And actually, I think the only mapfiles called directly are in

/usr/local/geomoose/gm3-demo-data/*/*.map

The top level ones are only referenced by INCLUDE statements.

[sdlime]

I'll supply a patch to the MapServer install script ASAP.

[kalxas]

Pull request merged:

​https://github.com/OSGeo/OSGeoLive/pull/313

but problem persists

[sdlime]

Ugh... Looking at the screenshots the problem is the /./ in the mapfile path. The value for MS_MAP_PATTERN is explicitly blocking that (and so will subsequent versions of MapServer by default).

• /usr/local/geomoose/gm3-demo-data/demo/./parcels/parcels.map - FAILS
• /usr/local/geomoose/gm3-demo-data/demo/parcels/parcels.map - WORKS

I don't know how those paths are built within GeoMoose.

kalxas, what's the easiest way for me to fire up a local version of OSGeoLive with this issue?

--Steve

[jimk]

This is coming from the GeoMoose Mapbooks:

/usr/local/geomoose/gm3-examples/htdocs/desktop/mapbook.xml
/usr/local/geomoose/gm3-examples/htdocs/mobile/mapbook.xml

Lines like:

<file>./demo/scalebars/scalebar_miles.map</file>

[sdlime]

Is it reasonable to remove the ./ or do I need to figure out a different expression that allows /./ in the path?

[jimk]

With this change in MapServer, I really want to move the GeoMoose to the /datasets/scalebar_miles.map?... style URLs (which can protect files and set the environment variables properly for each file using Apache directives) instead of /cgi-bin/mapserv?map=..., but I don't have time to do that before the OL14 release.

[jimk]

I think the ./ can be removed from the mapbooks (sed???).

[jimk]

I'm not sure if this misses something, but this is what I used on demo.geomoose.org.

SetEnv MS_MAP_PATTERN "^\/srv\/demo\/src\/3\.0\/gm3-demo-data\/(\./){0,1}([^\.][_A-Za-z0-9\-\.]+\/{1})*([_A-Za-z0-9\-\.]+\.(map))$"

[sdlime]

Reasonable hint. I think I can just do something like:

"^{/usr/local/(mapserver specific path)|(geomoose specific path)/([}\.][_A-Za-z0-9\-\.]+\/{1})*([_A-Za-z0-9\-\.]+\.(map))$"

So much easier in recently released version(s). More soon...

--Steve

[sdlime]

Just submitted another pull request that should allow /./ in paths. --Steve

​https://github.com/OSGeo/OSGeoLive/pull/314

[kalxas]

Thanks, building a new iso to test and will provide feedback.

[kalxas]

The error persists. Please test using the latest iso:

​http://aiolos.survey.ntua.gr/gisvm/dev/osgeolive-nightly-build285-amd64-3f5fd9c-master.iso

[sdlime]

Have latest ISO running locally as a VM. Will give it another shot...

[sdlime]

Ok, one more try. See ​https://github.com/OSGeo/OSGeoLive/pull/315. Tested against the nightly ISO... --Steve

[kalxas]

Confirmed fixed in build 287

Thanks all!