Fork me on GitHub
#pedestal
<
2024-03-27
>
phill10:03:58

Interesting observation "if a handler returns a partial response map, Pedestal accepts it as is, even if missing the :header"

phill10:03:06

P.S. It can be hard to tell whether "allows...to omit headers" is the problem or the solution. Reading the details, I infer a pattern of naming enhancement issues as if to complete the sentence, "effective with this patch, the system ___"? If the house were on fire, the enhancement might be titled, "doesn't burn"?

Ben Kamphaus14:03:09

also been burned by this several times

hlship05:03:03

Yes, it’s an odd mix of excessive rigor in one place — detecting the :response map to terminate the interceptor chain) — vs. excessive slack elsewhere: ignoring a :response map that doesn’t match the expectation (which defies the intent of the developer), and allowing pretty much anything from a handler because the interceptor chain terminates when it runs out of interceptors.

Ray Stubbs15:03:51

I got hit by this the other day too.

hlship18:04:32

It’s a pretty obvious deficiency, should have been addressed years ago. I suspect it’s a case of being too close to the problem.