Skip to content

Remove Google reCAPTCHA#33

Merged
ewalid merged 3 commits intomasterfrom
remove/recaptcha
Apr 6, 2026
Merged

Remove Google reCAPTCHA#33
ewalid merged 3 commits intomasterfrom
remove/recaptcha

Conversation

@ewalid
Copy link
Copy Markdown
Owner

@ewalid ewalid commented Apr 6, 2026

Summary

  • Remove all Google reCAPTCHA verification from the backend and frontend
  • Fixes translation failures for users on rosetta.study due to domain not being registered in the reCAPTCHA console

Changes

  • Backend: Removed verify_recaptcha(), RECAPTCHA_SECRET_KEY, and recaptcha_token params from /translate and /translate-stream
  • Frontend: Deleted Recaptcha.tsx / Recaptcha.css, removed all reCAPTCHA state/logic from TranslateForm.tsx, cleaned up types/index.ts and api/client.ts
  • Dependencies: Uninstalled react-google-recaptcha and @types/react-google-recaptcha
  • Env: Removed VITE_RECAPTCHA_SITE_KEY

Test plan

  • Verify translation works end-to-end without reCAPTCHA errors
  • Confirm no reCAPTCHA widget appears in the UI

🤖 Generated with Claude Code

ewalid and others added 3 commits April 6, 2026 11:30
- Remove reCAPTCHA verification from /translate and /translate-stream endpoints
- Delete Recaptcha.tsx and Recaptcha.css components
- Remove recaptchaToken from TranslateRequest type and API client
- Uninstall react-google-recaptcha and @types/react-google-recaptcha
- Remove VITE_RECAPTCHA_SITE_KEY from frontend env

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@ewalid ewalid merged commit 351bbb7 into master Apr 6, 2026
4 checks passed
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.

1 participant