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

Заголовки сессии имеют приоритет над заголовками передаваемых как параметры #121

Open
denisdert opened this issue Mar 6, 2023 · 2 comments · May be fixed by #122

Comments

@denisdert
Copy link

denisdert commented Mar 6, 2023

Пытаюсь передать заголовок "Accept".

настройкиПодключения_FacadeNet.НовыеПараметры.Заголовки.Вставить("Accept", "application/octet-stream")
ответHTTP	= FacadeCore_HTTPApi.Post(Url, , настройкиПодключения_FacadeNet.НовыеПараметры, FacadeCore_НастройкиПовтИсп.ТекущаяСессия());

Вместо этого передается

*/*

После изучения исходников было выяснено, что это значение устанавливается функцией

СоздатьСессию()

И в функции ПодготовитьЗапрос идет конструкция вида

Заголовки = Объединить(Скопировать(ЗаголовкиИзДополнительныхПараметров), Сессия.Заголовки); 

где затирается заголовок из параметров

Обошел только принудительным созданием сессии и удалением из нее данного заголовка. Если сессию принудительно не создавать, то она создается по умолчанию через вызов функции

ТекущаяСессия = ТекущаяСессия(Сессия);

Это ожидаемое поведение или ошибка?

Воспроизводится на 2.4.8 и 2.4.7.

@leemuar
Copy link
Collaborator

leemuar commented Jan 15, 2024

Баг привнесен изменением в функции Дополнить(): f0d37ae#diff-e3d8fad74c9a9d0f441bc9570022975306482a8c8f2b08091406d3bc8b197608R3321

@yuriipatlasov
Copy link

версия 2.6.0 все равно приоритет из Заголовков по умолчанию.
Заголовки.Вставить("Accept-Encoding", "gzip");
по умолчанию отключить сжатие ответа.
Все равно уходит этот заголовок.

Это проявляется на методе PUT, POST.

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