search

devicehive / esp8266-firmware

DeviceHive esp8266 firmware. Control hardware via clouds with DeviceHive!
MIT License
166 stars 50 forks source link

Authorization header check only in one variant #42

Open team55 opened 6 years ago

team55 commented 6 years ago

модуль httpd.c В rest запросах заголовок Authorization проверяется только на вариант 'Authorization', тогда как многие клиенты преобразуют его в нижний регистр. К примеру так делает react-native при работе через библиотеку okhttp, и на это нельзя повлиять.. Разный регистр заголовков вполне соответствует стандарту https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2

Логичнее было бы и проверять заголовки без привязки к регистру (либо преобразовать верхний-нижний)

team55 commented 6 years ago

https://github.com/facebook/react-native/issues/16988 думал проблема в ReactNative

Nikolay-Kha commented 6 years ago

Да... по спекам хедеры могут быть в любом регистре. Поправим. Спасибо.

Nikolay-Kha commented 6 years ago

Исправлено. Дайте знать если нужна бинарная сборка.