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.

Sign up for GitHub

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

Jump to bottom

feat: anti-bot filtering with reCAPTCHA v3 #137

Draft
wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

feat: anti-bot filtering with reCAPTCHA v3 #137

wants to merge 26 commits into from

Conversation

privilegemendes
Copy link
Contributor

@privilegemendes privilegemendes commented Dec 6, 2023 •
edited
Loading

Description:

This pull request introduces bot filtering functionality to enhance the security of our application by integrating reCAPTCHA v3. Bot filtering is crucial for preventing automated attacks and ensuring a better user experience.

Related Issues:

Integrate anti-bot filter #118

Changes Made:

  • Integration of reCAPTCHA v3:
  • Added reCAPTCHA v3 library to the project dependencies.
  • Configured reCAPTCHA site key and secret key in the application settings.
  • Implemented server-side verification logic to validate reCAPTCHA tokens via trpc.
  • Frontend Integration:
  • Updated start game button to trigger reCAPTCHA execution.
  • Handled reCAPTCHA response on the client side for enhanced user feedback.

@privilegemendes privilegemendes self-assigned this Dec 6, 2023
@privilegemendes privilegemendes linked an issue Dec 6, 2023 that may be closed by this pull request
Integrate anti-bot filter #118
Open
3 tasks
@privilegemendes privilegemendes changed the title Feature: Implement Bot Filtering with reCAPTCHA v3 Feature: Implement Anti-Bot Filtering with reCAPTCHA v3 Dec 6, 2023
@silimarius silimarius changed the title Feature: Implement Anti-Bot Filtering with reCAPTCHA v3 feat: anti-bot filtering with reCAPTCHA v3 Dec 7, 2023
ramidecodes
ramidecodes suggested changes Dec 7, 2023
View reviewed changes
src/containers/reCaptcha/utils.ts Outdated Show resolved Hide resolved
src/server/api/routers/recaptcha.ts Outdated Show resolved Hide resolved
src/service/use-reCaptcha.ts Outdated Show resolved Hide resolved
silimarius
silimarius requested changes Dec 7, 2023
View reviewed changes
Copy link
Collaborator

@silimarius silimarius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget to fix the pnp thing

src/components/menu-dialog/menu-dialog.tsx Outdated Show resolved Hide resolved
src/components/menu-dialog/menu-dialog.tsx Outdated Show resolved Hide resolved
src/components/menu-dialog/menu-dialog.tsx Show resolved Hide resolved
src/components/menu-dialog/menu-dialog.tsx Show resolved Hide resolved
src/containers/home/home.tsx Outdated Show resolved Hide resolved
src/pages/index.tsx Outdated Show resolved Hide resolved
src/pages/index.tsx Outdated Show resolved Hide resolved
src/server/api/routers/recaptcha.ts Outdated Show resolved Hide resolved
src/server/api/routers/recaptcha.ts Show resolved Hide resolved
src/server/api/routers/recaptcha.ts Outdated Show resolved Hide resolved
ramidecodes
ramidecodes suggested changes Dec 13, 2023
View reviewed changes
deployment/staging/deployment.yaml Show resolved Hide resolved
src/containers/home/home.tsx Outdated Show resolved Hide resolved
src/containers/home/home.tsx Outdated Show resolved Hide resolved
src/server/api/routers/recaptcha.ts Outdated Show resolved Hide resolved
Privilege Mendes added 5 commits December 15, 2023 10:47
@ramidecodes ramidecodes marked this pull request as draft January 17, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@silimarius silimarius silimarius requested changes

@ramidecodes ramidecodes ramidecodes requested changes

@Mautjee Mautjee Awaiting requested review from Mautjee

Requested changes must be addressed to merge this pull request.

Assignees

@privilegemendes privilegemendes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Integrate anti-bot filter
3 participants
@privilegemendes @ramidecodes @silimarius