[PATCH] HTTP/2: reject HTTP/2 requests without ":scheme" pseudo-header
Maxim Konovalov
maxim at nginx.com
Thu Mar 30 08:14:33 UTC 2017
Hi Piotr,
On 30/03/2017 04:01, Piotr Sikora via nginx-devel wrote:
> Hey Valentin,
>
>> IMHO it's not a good idea to combine style fixes with behavior changes.
>> Behavior changing commits are occasionally reverted.
>
> Fair enough, I'll update both patches shortly.
>
>> That's why it's still TODO (in other words intentionally skipped).
>> We discussed it with QA and decided to be more tolerant here.
>
> I disagree. Forgiving implementations that allow broken clients to
> seemingly "work", even when said clients are not obeying the
> specification, are the reason why we have broken clients in the first
> place.
>
How does google.com as a service behave with such clients?
--
Maxim Konovalov
More information about the nginx-devel
mailing list