EXCEPTIONS=application/Fvnd.ogc.se_inimage is not respected

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.6.3
  • Fix Version/s: 1.7.0-beta2
  • Component/s: WMS
  • Labels:
    None

Description

Trying this request (with a wrong cql filter):

http://localhost:8080/geoserver/wms?WIDTH=800&SRS=EPSG%3A4326&STYLES=&HEIGHT=317&LAYERS=topp%3Astates&FORMAT=image%2Fpng&SERVICE=WMS&VERSION=1.1.1&REQUEST=GetMap&EXCEPTIONS=application%2Fvnd.ogc.se_inimage&CQL_FILTER=states.1&BBOX=-145.489487296875,17.49445597548828,-46.21178370312501,56.83324602451172

results in an exception reported in xml, whilst the EXCEPTIONS attribute requested it to be embedded in the image. I guess somehow the dispatcher should make an exception for WMS and let it handle the exception in a different way. Justin, do we have any extension point in the dispatching system to allow for this kind of service specific exception handling?

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Chris Holmes added a comment -

Was it ever respected? I'm not convinced I've ever seen it actually return an image as an exception, though I never actually tested it closely. There's just sometimes when I thought it would be there and it wasn't.

Show
Chris Holmes added a comment - Was it ever respected? I'm not convinced I've ever seen it actually return an image as an exception, though I never actually tested it closely. There's just sometimes when I thought it would be there and it wasn't.
Hide
Permalink
Andrea Aime added a comment -

Hmmm... right, I did not check. I frankly never minded this much about this kind of exception reporting since it's not the default, but I realized how important it is when adding filtering abilities in the improved map preview, and noticed how you don't get any kind of feedback from OL (the service exception are not being reported in any way that I can see).
On the other side, I'm sure I've seen this kind of error reporting in MapServer and it was helpful. So, I believe we should act on this one, regardless of this being a regression, or not.

Show
Andrea Aime added a comment - Hmmm... right, I did not check. I frankly never minded this much about this kind of exception reporting since it's not the default, but I realized how important it is when adding filtering abilities in the improved map preview, and noticed how you don't get any kind of feedback from OL (the service exception are not being reported in any way that I can see). On the other side, I'm sure I've seen this kind of error reporting in MapServer and it was helpful. So, I believe we should act on this one, regardless of this being a regression, or not.
Hide
Permalink
Chris Holmes added a comment -

I definitely agree, I think it would be quite helpful.

Show
Chris Holmes added a comment - I definitely agree, I think it would be quite helpful.
Hide
Permalink
Andrea Aime added a comment -

Details on how to change the pluggable exception handlers to support this use case here:
http://www.nabble.com/Exception-handlers-and-WMS-exceptions-reported-in-application-vnd.ogc.se_inimage-format-td17430202.html

Show
Andrea Aime added a comment - Details on how to change the pluggable exception handlers to support this use case here: http://www.nabble.com/Exception-handlers-and-WMS-exceptions-reported-in-application-vnd.ogc.se_inimage-format-td17430202.html
Hide
Permalink
Andrea Aime added a comment -

Fixed on trunk. Changes are a wee bit extensive, don't know if it's worth porting back to 1.6.x series

Show
Andrea Aime added a comment - Fixed on trunk. Changes are a wee bit extensive, don't know if it's worth porting back to 1.6.x series
Hide
Permalink
Andrea Aime added a comment -

All these issues have been in "resolved" state for at least one month (many of them, much more than that). Since no one has reopened them, I'm mass-switching them to closed state. Reopen if you feel the issue has not been addressed properly.

Show
Andrea Aime added a comment - All these issues have been in "resolved" state for at least one month (many of them, much more than that). Since no one has reopened them, I'm mass-switching them to closed state. Reopen if you feel the issue has not been addressed properly.

People

  • Assignee:
    Andrea Aime
    Reporter:
    Andrea Aime
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: