- Issue created by @Anybody
- π©πͺGermany jan kellermann
jan kellermann β made their first commit to this issueβs fork.
- π©πͺGermany jan kellermann
I added the config for module drupal/recaptcha.
Do you need more config for other recaptcha modules?
Please review and feedback.
- Merge request !51#3491706: Add config for blocking reCaptcha. β (Merged) created by jan kellermann
- π©πͺGermany jan kellermann
I want to solve this without custom JS so we move the decoration from HTML-Parser to JS.
See #3491906
- π©πͺGermany jan kellermann
I changed the config-yaml. Please review together with #3491906 π Move decoration of custom elements from HTML-Helper to JS Active
- πΈπ°Slovakia poker10
Recaptcha has an option
use_globally
and then thehttps://www.recaptcha.net/recaptcha/api.js
is used instead ofhttps://www.google.com/recaptcha/api.js
. Could we also add the second script? - π©πͺGermany jan kellermann
I added the source
https://www.recaptcha.net/recaptcha/api.js
.Please test.
- π©πͺGermany jan kellermann
I tested in combination with #3491906 and the "globally"-option and it worked as expected.
We need a review for RTBC.
- π©πͺGermany jan kellermann
Testing can now be done directly with dev branch without further MRs.
- π©πͺGermany jan kellermann
Issue branch updated, MR testable without merge conflict.
- πΈπ°Slovakia poker10
I have tested the MR with a latest Drupal CMS and the new service seems to block recaptcha when "Use reCAPTCHA globally" is ON and also when it is OFF (so it blocks both https://www.google.com/recaptcha/api.js and https://www.recaptcha.net/recaptcha/api.js). Thanks!
-
jan kellermann β
committed 9a6f181c on 3.x
#3491706: Add config for blocking reCaptcha.
-
jan kellermann β
committed 9a6f181c on 3.x
- π©πͺGermany jan kellermann
Thank you very much for proposal, testing and feedback.
I merged the code.