NotificationsYou must be signed in to change notification settings
Fork9.7k
Star30.8k

[Validator] Remove checkDNS option in Url#31681

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

nicolas-grekas merged 1 commit intosymfony:masterfromro0NL:validator3

May 29, 2019

Merged

[Validator] Remove checkDNS option in Url#31681

nicolas-grekas merged 1 commit intosymfony:masterfromro0NL:validator3

May 29, 2019

Conversation

Copy link

Contributor

ro0NL commentedMay 29, 2019

Q	A
Branch?	master
Bug fix?	no
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets	#...
License	MIT
Doc PR	symfony/symfony-docs#...

See#25516

carsonbot added Status: Needs Review Validator labels

May 29, 2019

[Validator] Remove checkDNS option in Url

885703f

nicolas-grekas added this to the5.0 milestone

May 29, 2019

Copy link

Member

nicolas-grekas commentedMay 29, 2019

Thank you@ro0NL.

nicolas-grekas merged commit885703f intosymfony:master

May 29, 2019

nicolas-grekas added a commit that referenced this pull request

May 29, 2019

minor#31681[Validator] Remove checkDNS option in Url (ro0NL)

52756a5

This PR was merged into the 5.0-dev branch.Discussion----------[Validator] Remove checkDNS option in Url| Q             | A| ------------- | ---| Branch?       | master| Bug fix?      | no| New feature?  | no| BC breaks?    | no     <!-- seehttps://symfony.com/bc -->| Deprecations? | no| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->| Fixed tickets | #...   <!-- #-prefixed issue number(s), if any -->| License       | MIT| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->See#25516Commits-------885703f [Validator] Remove checkDNS option in Url