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

Complete Turnstile rollout #830

Open
HappyNTH opened this issue Oct 9, 2024 · 0 comments
Open

Complete Turnstile rollout #830

HappyNTH opened this issue Oct 9, 2024 · 0 comments
Labels
enhancement Relating to new features prio:medium Medium priority (think 'this term')

Comments

@HappyNTH
Copy link
Member

HappyNTH commented Oct 9, 2024

Turnstile has been implemented on the register mutation but should be rolled out to more mutations. The process is fairly simple (requiring the turnstile mutation to be mixed in with the relevant mutation, as register has been), but should be implemented once registration has been live for a little while in case any problems arise.

It should definitely be implemented on the following mutations:

  • Resending verification emails
  • Changing passwords
  • Changing emails

In addition, it should also be implemented for the login mutation. However, since manually triggering the login mutation is sometimes useful without a Turnstile token, we should also implement some override token that can be passed instead (stored in a .env file) to enable this (and probably all) mutation(s) to be manually triggered without needing to spoof Turnstile.

@HappyNTH HappyNTH added enhancement Relating to new features prio:medium Medium priority (think 'this term') labels Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Relating to new features prio:medium Medium priority (think 'this term')
Projects
Status: Medium Priority
Development

No branches or pull requests

1 participant