Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Authorization header check only in one variant #42

Open
team55 opened this issue Nov 28, 2017 · 3 comments
Open

Authorization header check only in one variant #42

team55 opened this issue Nov 28, 2017 · 3 comments
Labels

Comments

@team55
Copy link

team55 commented Nov 28, 2017

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

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

@team55
Copy link
Author

team55 commented Nov 28, 2017

facebook/react-native#16988 думал проблема в ReactNative

@Nikolay-Kha
Copy link
Contributor

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

@Nikolay-Kha
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants