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

Feat: add Google reCAPTCHA #7125

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

kasir-barati
Copy link

@kasir-barati kasir-barati commented Nov 7, 2024

Pull Request Requirements:

  • Provide a brief description of the changeset:

    Here I've created a new component called GoogleRecaptcha. By adding this component you'll get a recaptcha token from Google's API.

    Why should we take this seriously and address it? Although it's a non-functional requirement, it's invaluable because it prevents users from suffering from brute-force attacks.

    Closes: Feat: add bruteforce protection #6570.

  • Include a screenshots if applicable:

    ATM it is not completely functional since the backend part is missing but in general it won't be something that's gonna interrupt user's normal interactions with the system.

  • Ensure that the changeset adheres to the DCO guidelines.

    It does.

What I did not manage to get a hold of, the backend part of this app (specifically talking, I guess it should be hardcoreeng/account). I need to just add the functionality over there and reCAPTCHA will be finished. These steps needed to be taken:

  1. Send a req to Google's API to assess the performed action and receive a score.
  2. Interpret the score and based on the interpretation decide to whether you should allow the actions to proceed or should it be rejected.

Love to hear about your thoughts.

Huly®: UBERF-8685

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.

Feat: add bruteforce protection
1 participant