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

Adding full support for reCAPTCHA V3 and automatic binding to challenges #8

Merged
merged 18 commits into from
Apr 22, 2022

Conversation

jooni91
Copy link
Member

@jooni91 jooni91 commented Apr 22, 2022

The PR contains several new features and updates to existing ones.

Added:

  • An Action property to the ValidateRecaptchaFilter and the ValidateRecaptchaAttribute, for reCAPTCHA V3 action validation. (eebb309)

  • A default callback script is added to the bottom of the body when automatically binding the challenge to an element. (ca3d8a3)

  • New FormId property to RecaptchaInvisibleTagHelper, which should be set when automatically binding to a challenge. (ca3d8a3)

  • A new RecaptchaV3TagHelper for automatic binding of V3 challenges. (d5fef5d)

  • Added support for NETCOREAPP3.1, NET5, and NET6 to target frameworks. (7e5a4f4)

Updated:

  • XML documentation was updated for some classes, methods, and properties.
  • The RecaptchaInvisibleTagHelper now supports automatic binding to the challenge.
  • The RecaptchaScriptTagHelper now fully supports reCAPTCHA V3, so you may now make use of automatic or explicit rendering. (c771098)

Removed:

  • Dropped support NETCOREAPP2.1 and NETCOREAPP3.0 from target frameworks. (7e5a4f4)

Other:

Did several other updates and improvements to existing code and documentation. Also updated the NuGet metadata.

Upgrading support for netcoreapp3.0 to netcoreapp3.1.
Adding support for net5.0 and net6.0.
…rameworks into one item group. Bumping dependencies to the latest versions.
…which will be added if no callback is set and a form id is provided instead.

This makes usage of the invisible recaptcha tag even simpler.
…s there is not really a reason to make it accessible to the public.

Added a null check to the constructor.
…in the ValidateRecaptcha attribute. This will then also validate that the action does match the expected result.
@jooni91 jooni91 added the enhancement New feature or request label Apr 22, 2022
@jooni91 jooni91 merged commit 90f258e into master Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant