bcosca / fatfree

A powerful yet easy-to-use PHP micro-framework designed to help you build dynamic and robust Web applications - fast!

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Question] HTTP/1.1, HTTP/2 and HTTP/3 support

adeko opened this issue · comments

commented

I've noticed some protocol checks for HTTP/1.1.

web.php
&& preg_grep('/HTTP\/\d\.\d 200/',$response['headers'])
preg_grep('/HTTP\/1\.\d 3\d{2}/',$headers) &&
if (preg_match('/HTTP\/1\.\d 304/',

ws.php
if (preg_match('/^(\w+)\s(.+)\sHTTP\/1\.\d$/',
'HTTP/1.1 400 Bad Request'.$EOL.

Etc.

[Question] We already have HTTP/1.1, HTTP/2 and HTTP/3. Should this code be updated?

E.g.
preg_grep('/HTTP\/\d\.\d 200/',$response['headers'])
replace with
preg_grep('/HTTP\/[\d\.]{1,3} 200/',$response['headers'])