search

Lansoweb / LosReCaptcha

PHP module for using the ReCaptcha v2 system from Google
MIT License
3 stars 4 forks source link

ReCaptcha multi-instance #7

Open esprit-dev opened 7 years ago

esprit-dev commented 7 years ago

Hi,

I'm using your ReCaptcha module and it works pretty good :) But i had to find a workaround for keeping out conflits when using multiple form with captcha in the same page, as clones, which have no ID setted on captcha element. In this case, name of the element is taken into account, and there is conflicts betweens same forms.

Could you give me your feedback ? can you upgrade your module for multi-instance usage ? I can give you my patch file if necessary, that can help you to understand my request.

Thx

Lansoweb commented 7 years ago

@esprit-dev Hi! Sorry for the delay! Yes, paste your solution, also, i'll check with the official zend recaptcha (they releases a new version with v2 support this month). And thanks for your support!

Marc-pi commented 7 years ago

i'll check with the official zend recaptcha (they releases a new version with v2 support this month).

yes, invisible recaptcha is needed too :-)

Lansoweb commented 7 years ago

@Marc-pi True, but it's not released yet :-) For others, that's the invislbe recaptcha

Lansoweb commented 7 years ago

Multiple recaptchas is possible through the "explicit" mode as in documentation. Will work on that.

Marc-pi commented 7 years ago

yep, @esprit-dev work is there https://github.com/Marc-pi/pi/tree/develop/lib/vendor/LosReCaptcha i think we used this explicit method too, see https://github.com/Marc-pi/pi/commit/f5e6dba6241b91256d58e9c5123e2536ce412dd4

Lansoweb commented 7 years ago

@Marc-pi Great! Thanks, will work on an adaptation for this module soon. Nice work.

Lansoweb commented 7 years ago

@Marc-pi Hi! The invisible captcha is GA. Will add it to the lib soon.

Marc-pi commented 7 years ago

yes, i've noticed also, hope to see the lib improvements soon (multi-instance and invisible)

Marc-pi commented 7 years ago

we fixed one W3C error here https://github.com/pi-engine/pi/commit/701d4206d58a104ab11d3bb3142beec70e0a814d

Marc-pi commented 7 years ago

@Lansoweb did you find time to add the new invisible ?

Marc-pi commented 6 years ago

hello, any news on integrating invisible recaptcha ?

Lansoweb commented 6 years ago

Released 2.1.0 with invisible recaptcha. You can give it a try.

Marc-pi commented 6 years ago

good, we will try. for min Zend version, what did you set (we do not run 3x for now, we are on 2.4.10) ?

Lansoweb commented 6 years ago

I set only zend-form and zend-captcha:

"zendframework/zend-form": "^2.11",
"zendframework/zend-captcha": "^2.7"

So it should work with either zf version.

Marc-pi commented 6 years ago

ok, thx did you keep multi-instance (see above) ??