fix(api-form-builder): check both global and form recaptcha settings - Githubissues

webiny / webiny-js

Open-source serverless enterprise CMS. Includes a headless CMS, page builder, form builder, and file manager. Easy to customize and expand. Deploys to AWS.

https://www.webiny.com

Other

7.22k stars 589 forks source link

fix(api-form-builder): check both global and form recaptcha settings #4162

Closed Pavel910 closed 4 weeks ago

Pavel910 commented 4 weeks ago

Changes

This PR fixes a problem with Form Builder, related to reCaptcha. If you had reCaptcha enabled in your global Form Builder settings, it would apply to all forms, regardless of individual form settings. So even if you would disable recaptcha on a particular form, recaptcha checks would still kick in.

With this PR, we take per-form settings into account to determine whether recaptcha token verification is necessary.

How Has This Been Tested?

Manually.

Pavel910 commented 4 weeks ago

This code is pushed to dev branch, and we'll merge it into next from there.