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

Make sure error message shows in image captcha widget #1430

Merged
merged 34 commits into from
Oct 10, 2024

Conversation

forgetso
Copy link
Member

@forgetso forgetso commented Oct 3, 2024

  • error messages in procaptcha widget
  • allow onFailed to be set by clients
  • allow bundle example to run all captcha tests, including the server verify parts
  • adds cypress test for selecting incorrect images and ensures that custom onFailed function is fired
  • restrict API admin auth middleware to prosopo API paths in provider API
  • use environment variables in bundle demo instead of hard coded site keys
  • add user and site key logging for tracing errors
  • add register site key function to setup script
  • add unit test for renderLogic to ensure that callbacks are picked up from data tags
  • add unit tests for errorHandler to make sure parsing errors are passed through
  • add integration tests to make sure new errors are passed back
  • changes image captcha process to use timestamp signature in line with pow process
  • removes defunct user verify API path
  • fixes bug with vite config where defaultEnvironment could be set to something other than [development, staging, production]
  • add reset function to procaptcha-budle for re-rendering the procaptcha widgets
  • add onReset callback
  • pass an error JSON object in API errors, using translation keys as codes

Invalid Site Key

image

Site Key Not Registered

image

Failed to get Dataset

image

@forgetso forgetso merged commit 10e7a55 into staging Oct 10, 2024
4 checks passed
@forgetso forgetso deleted the fix-site-key-error-image-captcha branch October 10, 2024 13:09
@prosoponator prosoponator mentioned this pull request Oct 10, 2024
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.

2 participants