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

Выбран небезопасный способ передачи SERVER_KEY #449

Closed
sofieremi opened this issue Oct 28, 2022 · 5 comments
Labels
smartcaptcha Yandex SmartCaptcha

Comments

@sofieremi
Copy link

Ключ, токен и IP-адрес пользователя отправляются на проверку GET - запросом, что является очень небезопасным способом передачи секретной информации, так как она простым образом может быть скомпрометирована

def check_captcha(token):
resp = requests.get(
"https://captcha-api.yandex.ru/validate",
{
"secret": SMARTCAPTCHA_SERVER_KEY,
"token": token,
"ip": "<IP-адрес_пользователя>"
}

Этот пример вводит в заблуждение, так как метод POST лучше подходит для отправки данных, он более безопасный, обратите внимание на этот момент)

@kamkib kamkib added the smartcaptcha Yandex SmartCaptcha label Nov 3, 2022
@kamkib
Copy link
Contributor

kamkib commented Nov 3, 2022

Добрый день. Спасибо за замечание, взяли в работу.

@antonydevanchi
Copy link
Contributor

@kamkib допустим, и IP-адрес, и секрет от инстанса капчи — скомпрометировали.

И чего самого страшного можно с ними сделать? :)

@lxchurbakov
Copy link

Почему POST более безопасен, чем GET? Кем могут быть скомпрометированы данные, передающиеся по https?

@AlexJameson
Copy link
Collaborator

@sofieremi Здравствуйте! Команда разработки рассмотрела ваше замечание и начала работу над этой задачей.

@tatlush
Copy link

tatlush commented Oct 15, 2024

@sofieremi Все примеры в документации теперь приведены для POST-запросов: https://yandex.cloud/ru/docs/smartcaptcha/
Спасибо, что помогаете нам становиться лучше!

@tatlush tatlush closed this as completed Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
smartcaptcha Yandex SmartCaptcha
Projects
None yet
Development

No branches or pull requests

6 participants