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 · 1 comment · 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

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