Skip to content

MA-22157: fix header parser and pinned stable yarn version #68

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pasha-nikitin-2003
Copy link
Contributor

Изменения:

  • Изменен парсинг заголовков, раньше они склеивались в одну строку, из-за этого некоторые из них были установлены в неправильном формате.
  • Закреплен yarn 4 версии.
  • Исправлены ошибки типизации, вызванные прошлыми обновлениями.

@pasha-nikitin-2003 pasha-nikitin-2003 self-assigned this Apr 28, 2025
@pasha-nikitin-2003 pasha-nikitin-2003 requested a review from a team as a code owner April 28, 2025 17:13
@pasha-nikitin-2003 pasha-nikitin-2003 force-pushed the pavelnikitin/fix/parse-http-headers/MA-22157 branch from e802b27 to e750f09 Compare April 28, 2025 17:13
@pasha-nikitin-2003 pasha-nikitin-2003 force-pushed the pavelnikitin/fix/parse-http-headers/MA-22157 branch from e750f09 to 7789bca Compare April 28, 2025 17:15
@pasha-nikitin-2003 pasha-nikitin-2003 requested a review from a team as a code owner April 28, 2025 17:15
@nshvyryaev
Copy link

Нам нужно положить ярн в репу чтобы залочить его версию?

nshvyryaev
nshvyryaev previously approved these changes Apr 29, 2025
MrYadro
MrYadro previously approved these changes Apr 29, 2025
Pronin-vk
Pronin-vk previously approved these changes May 6, 2025
@pasha-nikitin-2003 pasha-nikitin-2003 force-pushed the pavelnikitin/fix/parse-http-headers/MA-22157 branch 8 times, most recently from 33b99cc to b8dd28d Compare May 7, 2025 14:05
@pasha-nikitin-2003 pasha-nikitin-2003 force-pushed the pavelnikitin/fix/parse-http-headers/MA-22157 branch from b8dd28d to 531bb92 Compare May 7, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Заголовок set-cookie некорректно устанавливается
6 participants