David Precious

This is still reproducible with 7.0.7 - for e.g. providing some invalid input to force an error: ``` [davidp@columbia:~]$ mapserv QUERY_STRING="totally invalid" && echo -e "\nReturned: $?" Content-Type: text/html MapServer...

I'd be inclined to say that, if the merge conflicts can be fixed, this ought to be merged, and the doco as it then stands re-reviewed and improved further, using...

Right - this one needs addressing - good doco is incredibly important, and the great work done in this PR is sitting unused! I'm of the opinion that, even if...

As I see it, renaming the dancer2 script isn't a fix, it's a bodgy workaround for a broken system. Fixing the broken system would be far better, or if a...

Perhaps. A default implementation which helps save people from not having considered the possibility of accidentally giving away sensitive information seems a worthwhile feature to me still - but being...

Will see what I can do :)

This looks pretty sane and useful to me - thanks for the contribution! Would like another core dev to review as well though, in particular the addition of Devel::StackTrace as...

Bugger. Looking in to this one today. It's weird, as the PR included tests which ensure that response->body works as it should.

Right - if you're POSTing raw content (`application/json` or `application/octet-stream`), then `HTTP::Body::OctetStream` stores it into a temp file stored in `$self->body` which we can read from. However, if you've posted...

Ah - but we won't know the type of request until we've passed enough data to `HTTP::Body`, and the chunks of data we read and pass to `$self->{_http_body}->add` will not...