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

Восстановил авторизацию через логин/пароль #1247

Merged
merged 2 commits into from
Aug 26, 2022

Conversation

Selive
Copy link
Contributor

@Selive Selive commented Aug 22, 2022

Список изменений

Что было

Для запроса авторизации нудно было указать Content-Type : application/x-www-form-urlencoded и по хорошему использовать заголовки origin и referer.

Обязательно выполните следующие пункты:
  • Проверьте что ваш код не содержит конфликтов, и исправьте их по необходимости
  • Напишите тесты, и обязательно проверьте что не падают другие.
Внимание! Pull Request'ы с не пройдёнными тестами не принимаются

Для метода авторизации нужны были POST заголовки
@inyutin-maxim inyutin-maxim merged commit 1c43ed1 into vknet:develop Aug 26, 2022
@inyutin-maxim
Copy link
Member

@Selive спасибо за проделанную работу. Добавлено к следующему релизу. Будет в начале сентября. Я пока в отпуске

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

Successfully merging this pull request may close these issues.

2 participants