MapServer / MapServer-import

3 stars 2 forks source link

grouped and named layers with REQUIRES cause Internal Server Error #1283

Open tbonfort opened 12 years ago

tbonfort commented 12 years ago

Reporter: yuri0002@smis.iki.rssi.ru Date: 2005/03/09 - 17:21

LAYER
    NAME 'L1'
    GROUP 'GR'
    REQUIRES '[L3]'
    ...
END
LAYER
    NAME 'L2'
    GROUP 'GR'
    REQUIRES '[L4]'
    ...
END
Request of layer "GR" will cause 500 Internal Server Error ("Premature end of 
script headers: /usr/local/www/cgi-bin/mapserv") unless NAME or REQUIRES is 
commented out for one of layers.
There is no such bug in version 4.2
tbonfort commented 12 years ago

Author: sdlime Date: 2005/03/09 - 18:04

This a good clue, thanks. The requires support was basically broken in 4.2 and 
only recently fixed. Looks to still be an issue though.

Steve
tbonfort commented 12 years ago

Author: sylvain.pasche@camptocamp.com Date: 2005/06/01 - 17:03

*** Bug 1309 has been marked as a duplicate of this bug. ***
tbonfort commented 12 years ago

Author: sdlime Date: 2005/06/01 - 18:13

I'll have a look at your patch. Thanks...

Steve
tbonfort commented 12 years ago

Author: sgillies@frii.com Date: 2005/08/25 - 16:35

This is a not minor regression in MapServer's capabilities to me. I have a
customer who can't upgrade a mapfile to MapServer 4.4 or 4.6 because of this bug.
tbonfort commented 12 years ago

Author: sdlime Date: 2005/08/25 - 16:52

That was a subtle kick in the pants. I'll address this one this evening (before 
any 4.6.1 release). The patch simply turns off the functionality which isn't a 
good fix.

I am suprised this is holding up upgrades since it didn't work at all in 4.4 
due to changes made by other developers...

Steve
tbonfort commented 12 years ago

Author: sgillies@frii.com Date: 2005/08/25 - 17:06

I meant that they can't upgrade *to* 4.4 or 4.6. Is requires completely broken,
then? If so, my users can probably accept that and commit to writing the
workarounds.
tbonfort commented 12 years ago

Author: sdlime Date: 2005/08/25 - 17:44

Lemme try to fix tonite before starting on workarounds. It's not completely 
broken in 4.6, but the group issue pointed out in this bug causes problems in 
certain instances. I'm using REQUIRES in production apps with no problem.

Steve
tbonfort commented 12 years ago

Author: sdlime Date: 2005/08/27 - 07:12

Ok, I found the problem in maputil.c - stupid on my part. The fix was easy but I
had to disable the [raster] context option. There's no way, as it was setup, to
avoid possible recursion errors with that. May need to expand that option into
the list of RASTER layers or something like that. Anyway, this did not make it
into 4.6.1, but we could do a 4.6.2 very soon. Can folks test on their end and
let me know how it goes?

Steve
tbonfort commented 12 years ago

Author: tims@commenspace.org Date: 2005/09/17 - 21:45

I'm rewriting my mapscript workaround for this bug and wanted to get
confirmation on something.

If I have a layer with REQUIRES ![raster], and the raster layer is DEFAULT, even
if the map scale is <= MINSCALE or > MAXSCALE for the raster, my layer (with
REQUIRES ![raster]) will not display.  Correct?

(I need to write a workaround because I have scale dependent raster layers with
alpha bands that aren't necessarily "background" layers.)

Thanks,
Tim
tbonfort commented 12 years ago

Author: sdlime Date: 2005/09/21 - 05:53

I don't think I will re-implement the raster option. Too many problems. Not a
bad idea but not universally applicable. Marking as fixed for 4.8. I can
backport if there is sufficient interest.

Steve