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

Decouple HTTP client and validator so we can verify responses outside Forms #7

Merged
merged 3 commits into from
May 24, 2024

Conversation

ikvasnica
Copy link
Contributor

Hi,

I'm sending this PR as a result of our use case where we implemented the package into Symfony Forms. But then we needed to use Cloudflare Turnstile also within JavaScript part of the application where we don't use Symfony Forms, however we still need to validate Turnstile response using PHP backend.

The goal was to decouple HTTP client and Symfony Form Validator parts.
Plus I added a unit test for this part.

@matthieu2607
Copy link
Collaborator

Hi @ikvasnica,
Thank you for your superb work!
I'll have a look at your changes before releasing
Best regards

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@matthieu2607 matthieu2607 merged commit 65dd4b6 into Pixel-Open:develop May 24, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants