[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