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

[Feature] Handling g-recaptcha errors #28

Open
Streemo opened this issue Nov 18, 2015 · 0 comments
Open

[Feature] Handling g-recaptcha errors #28

Streemo opened this issue Nov 18, 2015 · 0 comments

Comments

@Streemo
Copy link

Streemo commented Nov 18, 2015

Current behavior (pseudocode):

if (captcha passed)
  redirect back to original page
else
  response.end('Captch verification failed!') //plain text
  return true;

Improved behavior (pseudocode):

if (captcha passed)
  redirect back to original page
else
  var thereWasAProblemHtml = Assets.getText('googleFailed.html')
  response.write(thereWasAProblemHtml) //html
  setTimeoutForBlockIpForMilliSeconds
    redirect back to original page
    res.end()
    return true;

This way, the user is not shown plain text when google messes up, and instead Sikka has a fallback (redirect in blockIpFor milliseconds). Currently, Sikka will end the response with plain text, forcing the user to manually try again.

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

No branches or pull requests

1 participant