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

Update Recaptcha to v2 #468

Merged
merged 2 commits into from Apr 16, 2019
Merged

Update Recaptcha to v2 #468

merged 2 commits into from Apr 16, 2019

Conversation

@jesusbagpuss
Copy link
Contributor

@jesusbagpuss jesusbagpuss commented Oct 31, 2017

Resolves #354

This PR:

  • removes the V1 API calls
  • adds V2 API calls
  • updates phrases
  • adds an example cfg.d file

@phluid61 - also relates to https://github.com/QUTlib/eprints/tree/feature/recaptchav2

@prazetyo
Copy link

@prazetyo prazetyo commented Jan 19, 2019

Hi,
When I'm running this script on eprints 3.3.16 why there is an error like this :

JSON text must be an object or array (but found number, string, true, false or null, use allow_nonref to allow this) at /usr/share/eprints3/perl_lib/EPrints/MetaField/Recaptcha.pm line 128.\n

Maybe someone can help me? thank you.

@drn05r drn05r added this to the 3.3.17 milestone Apr 15, 2019
@drn05r drn05r mentioned this pull request Apr 15, 2019
@drn05r drn05r merged commit c6a553a into 3.3 Apr 16, 2019
@thisnugroho
Copy link

@thisnugroho thisnugroho commented Sep 1, 2021

Hello there, i still facing the same issue with this error

JSON text must be an object or array (but found number, string, true, false or null, use allow_nonref to allow this) at /usr/share/eprints3/perl_lib/EPrints/MetaField/Recaptcha.pm line 128.\n

i've done fetching the latest update of Recaptcha.pm then restarting apache2 and reload the eprints,but still not able to do registration,,

Maybe i did something wrong,can someone help me ?

Thanks..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants