Fhemweb Probleme mit lowercase HTTP Headern

Begonnen von HolyMoly, 16 Juli 2015, 13:05:58

Vorheriges Thema - Nächstes Thema

HolyMoly

Konkret ist es problematisch wenn man z.B. per nodejs request die jsonlist2 von einem per basic auth gesicherten fhem server holen will siehe https://github.com/request/request/issues/1677#issuecomment-121888100

Fhemweb besteht auf ein case sensitives "Authorization". Aber RFC 7230 erlaubt eigentlich: "Each header field consists of a case-insensitive field name followed by a colon (":"), optional leading whitespace, the field value, and optional trailing whitespace." Das ist natürlich irgendwie unbefriedigend  ::)

Könnte man vielleicht die FW_httpheader gleich title casen / normalisieren wenn sie reinkommen? Oder wie könnte man das sonst noch (mit minimalem Aufwand) angehen?

FHEM auf Raspi2 & Radxa Rock

rudolfkoenig

Ich habe jetzt etwas ungern Titlecaps eingebaut.
- Titlecaps, weil es an etlichen Stellen (auch in fremden Modulen) auf das Header so zugegriffen wird, und ich wollte moeglichst wenig aendern.
- Ungern, weil ich nicht sicher bin, ob das nicht irgendwelche Seiteneffekte hat.

Ich finde es doof, wenn das nodejs Modul die Header anders bauen muss, wie alle Browser, RFC hin oder her, es ist "asking for trouble". Hoffentlich verbringen sie viel Zeit mit dem Debuggen der Probleme.

HolyMoly

Vielen Dank dass du so schnell Abhilfe geschaffen hast.
Ich freue mich auf das Update morgen und hoffe dass die Nebeneffekte aus bleiben.
FHEM auf Raspi2 & Radxa Rock