Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Validator] Deprecate checkMX and checkHost on Email validator #28842

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

Merged
merged 1 commit into from
Oct 12, 2018

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Oct 12, 2018

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

@fabpot fabpot force-pushed the checkMX-checkMX-deprecation branch from d16af55 to b6fed05 Compare October 12, 2018 18:43
@fabpot fabpot force-pushed the checkMX-checkMX-deprecation branch from b6fed05 to 4c724be Compare October 12, 2018 18:48
@fabpot fabpot force-pushed the checkMX-checkMX-deprecation branch from 4c724be to c6009a0 Compare October 12, 2018 19:11
@fabpot fabpot merged commit c6009a0 into symfony:master Oct 12, 2018
fabpot added a commit that referenced this pull request Oct 12, 2018
…alidator (fabpot)

This PR was merged into the 4.2-dev branch.

Discussion
----------

[Validator] Deprecate checkMX and checkHost on Email validator

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | yes
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | fixes #27559 fixes #28665
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->

<!--
Write a short README entry for your feature/bugfix here (replace this comment block.)
This will help people understand your PR and can be used as a start of the Doc PR.
Additionally:
 - Bug fixes must be submitted against the lowest branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the master branch.
-->

Commits
-------

c6009a0 [Validator] deprecate checkMX and checkHost on Email validator
This was referenced Nov 3, 2018
@fabpot fabpot deleted the checkMX-checkMX-deprecation branch January 14, 2019 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Validator] Email validation false-positive due to relative host
4 participants