HTTPRequest - Response-Header-Case

SoftCreatR · January 30, 2020 at 10:43 PM

Ich möchte vorschlagen, dass die Keys im Response-Header-Array vereinheitlicht werden. Auch wenn ich bisher keine Probleme damit hatte, so ist mir zumindest heute bei Twitter aufgefallen, dass dort sämtliche Response-Header in Lowercase geschrieben sind, wogegen das auf anderen Seiten eher nicht so ist.

**Tim Düsterhus** · January 31, 2020 at 12:31 AM Official Post

Hallo,

das sollte bereits der Fall sein: https://github.com/WoltLab/WCF/bl….class.php#L428

SoftCreatR · January 31, 2020 at 12:35 AM

Tim Düsterhus

Och, ist ja erst 6 Jahre her Aber komisch ist das trotzdem:

PHP

$request = new HTTPRequest($url, [
    'method' => 'GET',
    'maxLength' => 512000,
    'timeout' => 5,
    'maxDepth' => 5
]);

$request->execute();
$reply = $request->getReply();

wcfdebug($reply['headers']);

Display More

https://www.woltlab.com

Code

Array
(
    [Date] => Thu, 30 Jan 2020 23:37:39 GMT
    [Content-Type] => text/html; charset=UTF-8
    [Transfer-Encoding] => chunked
    [Connection] => close
    [Set-Cookie] => wsc30_cookieHash=812c251df0f89bf7fff35f3d2efb9099907e9585; path=/; domain=.woltlab.com; secure; HttpOnly
    [Expires] => Mon, 26 Jul 1997 05:00:00 GMT
    [Last-Modified] => Thu, 30 Jan 2020 23:37:39 GMT
    [Cache-Control] => max-age=0, no-cache, no-store, must-revalidate
    [Pragma] => no-cache
    [Content-Encoding] => gzip
    [X-UA-Compatible] => IE=edge
    [X-Frame-Options] => SAMEORIGIN
    [Strict-Transport-Security] => max-age=31536000
    [X-Content-Type-Options] => nosniff
    [X-XSS-Protection] => 1; mode=block
    [CF-Cache-Status] => DYNAMIC
    [Expect-CT] => max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
    [Server] => cloudflare
    [CF-RAY] => 55d746672f0dd6fd-FRA
)

Display More

https://www.twitter.com

Code

Array
(
    [cache-control] => no-cache, no-store, must-revalidate, pre-check=0, post-check=0
    [connection] => close
    [content-encoding] => gzip
    [content-security-policy] => connect-src 'self' blob: https://*.giphy.com https://*.pscp.tv https://*.video.pscp.tv https://*.twimg.com https://api.twitter.com https://caps.twitter.com https://media.riffsy.com https://pay.twitter.com https://sentry.io https://ton.twitter.com https://twitter.com https://upload.twitter.com https://www.google-analytics.com https://app.link https://api2.branch.io https://bnc.lt https://vmap.snappytv.com https://vmapstage.snappytv.com https://vmaprel.snappytv.com https://vmap.grabyo.com https://mdhdsnappytv-vh.akamaihd.net https://mpdhdsnappytv-vh.akamaihd.net https://mmdhdsnappytv-vh.akamaihd.net https://smdhdsnappytv-vh.akamaihd.net https://smpdhdsnappytv-vh.akamaihd.net https://smmdhdsnappytv-vh.akamaihd.net https://rmdhdsnappytv-vh.akamaihd.net https://rmpdhdsnappytv-vh.akamaihd.net https://rmmdhdsnappytv-vh.akamaihd.net https://dwo3ckksxlb0v.cloudfront.net ; default-src 'self'; form-action 'self' https://twitter.com https://*.twitter.com; font-src 'self' https://*.twimg.com; frame-src 'self' https://twitter.com https://mobile.twitter.com https://pay.twitter.com https://cards-frame.twitter.com ; img-src 'self' blob: data: https://*.cdn.twitter.com https://ton.twitter.com https://*.twimg.com https://www.google-analytics.com https://www.periscope.tv https://www.pscp.tv https://media.riffsy.com https://*.giphy.com https://*.pscp.tv; manifest-src 'self'; media-src 'self' blob: https://twitter.com https://*.twimg.com https://*.vine.co https://*.pscp.tv https://*.video.pscp.tv https://*.giphy.com https://media.riffsy.com https://mdhdsnappytv-vh.akamaihd.net https://mpdhdsnappytv-vh.akamaihd.net https://mmdhdsnappytv-vh.akamaihd.net https://smdhdsnappytv-vh.akamaihd.net https://smpdhdsnappytv-vh.akamaihd.net https://smmdhdsnappytv-vh.akamaihd.net https://rmdhdsnappytv-vh.akamaihd.net https://rmpdhdsnappytv-vh.akamaihd.net https://rmmdhdsnappytv-vh.akamaihd.net https://dwo3ckksxlb0v.cloudfront.net; object-src 'none'; script-src 'self' 'unsafe-inline' https://*.twimg.com   https://www.google-analytics.com https://twitter.com  'nonce-NGUyYzI0OTgtYzE2Yi00ZmIwLTg5OGItZWZmNGNmYTExODAw'; style-src 'self' 'unsafe-inline' https://*.twimg.com; worker-src 'self' blob:; report-uri https://twitter.com/i/csp_report?a=O5RXE%3D%3D%3D&ro=false
    [content-type] => text/html; charset=utf-8
    [date] => Thu, 30 Jan 2020 23:38:11 GMT
    [expiry] => Tue, 31 Mar 1981 05:00:00 GMT
    [last-modified] => Thu, 30 Jan 2020 23:38:11 GMT
    [pragma] => no-cache
    [server] => tsa_o
    [set-cookie] => guest_id=v1%3A158042749124084955; Max-Age=63072000; Expires=Sat, 29 Jan 2022 23:38:11 GMT; Path=/; Domain=.twitter.com
    [strict-transport-security] => max-age=631138519
    [transfer-encoding] => chunked
    [vary] => Accept-Encoding
    [x-connection-hash] => 398b22916675f428c62a10c9062781e4
    [x-content-type-options] => nosniff
    [x-frame-options] => DENY
    [x-powered-by] => Express
    [x-response-time] => 193
    [x-xss-protection] => 0
)

Display More

---

Sonst wäre mir das gar nicht aufgefallen.

SoftCreatR · January 31, 2020 at 1:21 AM

Ich habe es jetzt noch ein paar Mal getestet und der Großteil aller getesteten Webseiten liefert die Header nicht in Lowercase aus. Auch unsere eigene nicht:

Code

Array
(
    [Date] => Fri, 31 Jan 2020 00:17:54 GMT
    [Content-Type] => text/html; charset=UTF-8
    [Transfer-Encoding] => chunked
    [Connection] => close
    [Set-Cookie] => jP9mKpy5t49NQXDN_cookieHash=152cbd52f75fe50bb702998406c272cf4c914a6c; path=/; domain=.softcreatr.com; secure; HttpOnly
    [Expires] => Mon, 26 Jul 1997 05:00:00 GMT
    [Last-Modified] => Fri, 31 Jan 2020 00:17:54 GMT
    [Cache-Control] => max-age=0, no-cache, no-store, must-revalidate
    [Pragma] => no-cache
    [X-UA-Compatible] => IE=edge
    [X-Frame-Options] => SAMEORIGIN
    [X-XSS-Protection] => 1; mode=block
    [X-Content-Type-Options] => nosniff
    [Referrer-Policy] => no-referrer
    [Content-Security-Policy] => default-src * data: blob: 'unsafe-eval' 'unsafe-inline'
    [Strict-Transport-Security] => max-age=31536000; includeSubDomains; preload
    [CF-Cache-Status] => DYNAMIC
    [Expect-CT] => max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
    [Alt-Svc] => h3-24=":443"; ma=86400, h3-23=":443"; ma=86400
    [Server] => cloudflare
    [CF-RAY] => 55d78157c8e559f4-VIE
    [Content-Encoding] => gzip
)

Display More

Und wenn wir gerade dabei sind: Können wir (zumindest ab 5.3) einen Integer als Status-Code zurück bekommen? Ist zwar kein Weltuntergang, dass dem aktuell nicht so ist, allerdings tappe ich immer in diese Falle und wundere mich am Ende, warum nichts geht.

**Tim Düsterhus** · January 31, 2020 at 10:38 AM Official Post

Hallo,

Du willst replyHeaders statt headers: https://github.com/WoltLab/WCF/bl….class.php#L593

Quote from SoftCreatR

Können wir (zumindest ab 5.3) einen Integer als Status-Code zurück bekommen?

Ohne es mir angeschaut zu haben: Aktuell ist es ein String oder worauf zielt die Frage ab?

SoftCreatR · January 31, 2020 at 10:44 AM

Aktuell ist es ein String, ja. Denn der Code wird mittels Regex extrahiert. Und bei Ganzzahlen erwarte ich eigentlich einen Integer. Wie gesagt, ist das nicht besonders wichtig, aber nice to have.

Und zwecks der Header: Danke, werde ich dann entsprechend umbauen.

**Tim Düsterhus** · January 31, 2020 at 10:49 AM Official Post

Hallo,

Quote from SoftCreatR

Aktuell ist es ein String, ja. Denn der Code wird mittels Regex extrahiert. Und bei Ganzzahlen erwarte ich eigentlich einen Integer. Wie gesagt, ist das nicht besonders wichtig, aber nice to have.

Das gleiche was für eine Änderung spricht (strikte Typen) spricht im Sinne der Kompatibilität streng genommen auch dagegen. Wenn du magst, dann mach einen PR mit der Änderung auf und man kann es dort ausdiskutieren.

**Alexander Ebert** · January 31, 2020 at 11:18 AM Official Post

Wir verwenden aktuell noch keine strikte Typisierung, auch weil PHP in den verbreiteten Versionen noch ziemlich inkonsistent ist, selbst PHP 7.4 hat noch Lücken aufzuweisen. Eine Änderung des Datentypes kommt daher aktuell nicht in Frage, diese würde potentiell mit der Kompatibilität im Hinblick auf einen expliziten String-Vergleich brechen. Es empfiehlt sich die Verwendung des impliziten Vergleiches nach Wert, statt nach Datentyp.

HTTPRequest - Response-Header-Case

Marcel Werk March 2, 2020 at 4:23 PM

Marcel Werk March 2, 2020 at 6:04 PM

Participate now!

Marcel Werk March 2, 2020 at 4:23 PM

Marcel Werk March 2, 2020 at 6:04 PM

Participate now!

Marcel Werk March 2, 2020 at 4:23 PM

Marcel Werk March 2, 2020 at 6:04 PM