-
Notifications
You must be signed in to change notification settings - Fork 112
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
"recaptcha" plugin incompatible with django-recaptcha>=2.0.0 #253
Comments
@barseghyanartur hi, what is your opinion about a fix for this ? Since django-recaptcha==1.4.0 has dropped support for Django<=1.11 but fobi still support it. Also the old ReCaptcha widget has been renamed ReCaptchaV2Checkbox but there is also a new widget ReCaptchaV3 introduced in 2.0.0 but this seems out of scope for the fix which is only about V2. What do you think ? |
I think it's time to drop everything below Django 2.2. If you have time to work on making |
Seems ok, first i did want to use the V3 but there seems to have some minor bugs for now, since V2 is solid enough i will focus on it so it's reliable. So i will look at this for next week, starting on 0.16.x for recaptcha 2.0.6 and V2. |
BTW, are you aware of invisible_recaptcha? I liked that one all the most. :) It should, however, not stop us improving |
Yeah i now about the invisible one, but never used it yet. By the way it seems this one should work, from the recaptcha diff i don't see any incompatible changes. |
Ping. |
@barseghyanartur Sorry for the delay. I've just tried with your last commit (b393446) from branch https://github.com/barseghyanartur/django-fobi/tree/v0.17.x and it works well, not any error when creating new form with a recaptcha field, updating it and using the form in frontend. Here is the environment for my tests:
By the way, i wanted to test the invisible recaptcha as you talked before, with the same environment in a new form and it does not succeed to validate the captcha field, i got this error in my form when submiting everything correctly:
As described from this plugin Readme i've added settings
And my frontend have jQuery v3.5.1 and did not see any Javascript error or Django error logs. |
Also you got some warnings from Python about deprecated
https://github.com/barseghyanartur/django-fobi/blob/master/src/fobi/base.py#L1553 However, feel free to close this issue if it's ok for you since original subject from this issue has been fixed. |
@sveetch: |
Bug Report
Expected behavior/code
fobi.contrib.plugins.form_elements.security.recaptcha
should be compatible with last django-recaptcha version.Actual Behavior
django-recaptcha release 2.0.0 has made some breakward incompatible refactoring on widget:
django-recaptcha/django-recaptcha@0a46ba8#diff-038fe6799d7292706ae4b851dd2dd64165d8018e8610bb25452f3457d562774b
This leads to an error on this plugin which try to import the old widget name "ReCaptcha":
https://github.com/barseghyanartur/django-fobi/blob/master/src/fobi/contrib/plugins/form_elements/security/recaptcha/base.py#L24
Steps to Reproduce
Environment
The text was updated successfully, but these errors were encountered: