Reject request with invalid Connection header format #1605
Conversation
0x42
changed the title
|
@essen do we need to improve anything in the PR? We need this fix badly. Regarding failed CI checks: I think this is unrelated to the changes. Correct me if I am wrong :) |
|
Nothing obvious just busy with other things. I'll try to schedule time to make a release and will handle the pending PRs at the same time. |
|
Please rebase onto current master so that CI runs properly. I am planning to do a release with this and other things soon. |
0x42
force-pushed
the
malformed_header_crash
branch
from
869fd34
to
72b9fa4
Compare
|
Unfortunately CI failed ((( |
|
The failure on OTP master is expected as there's a bug in OTP causing it. The one on 26.1 is not but likely a flake so I don't think there's any problem. Cheers. |
|
Merged with tweaks and an additional test for HTTP/1.0. Closing, thanks! |
Cowboy reject request with invalid Connection header format instead of throwing exception.