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

Email tweaks #988

Merged
merged 8 commits into from Sep 8, 2022
Merged

Email tweaks #988

merged 8 commits into from Sep 8, 2022

Conversation

samuelcolvin
Copy link
Member

@samuelcolvin samuelcolvin commented Nov 9, 2019

Change Summary

Some small and subtle tweaks to email validation. This will probably need to wait until v2 to be released.

As far as I know the following things have changed (but email-validator's tests aren't that clear, so I could be wrong):

  • whitespace is stripped from names, e.g. from ' Fred Smith <fred@example.com>' the name would now be 'Fred Smith', before it would have been ' Fred Smith'
  • the local part can be changed in the case of some strange encodings
  • the domain part can be changed in IDNA cases, e.g. xn--80ak6aa92e.com is changed to аррӏе.com

Also need to add a best error message done.

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • changes/<pull request or issue id>-<github username>.md file added describing change
    (see changes/README.md for details)

@samuelcolvin samuelcolvin added the deferred deferred until future release or until something else gets done label Nov 9, 2019
@codecov
Copy link

codecov bot commented Nov 9, 2019

Codecov Report

Merging #988 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #988   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          19     19           
  Lines        3287   3286    -1     
  Branches      650    650           
=====================================
- Hits         3287   3286    -1
Impacted Files Coverage Δ
pydantic/errors.py 100% <100%> (ø) ⬆️
pydantic/networks.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update caa270a...e5f8082. Read the comment docs.

tests/test_networks.py Outdated Show resolved Hide resolved
Copy link
Member Author

@samuelcolvin samuelcolvin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Experiment in reviews "please update".

tests/test_networks.py Outdated Show resolved Hide resolved
Copy link
Member

@hramezani hramezani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

I just left a small suggestion

Please update

@samuelcolvin samuelcolvin enabled auto-merge (squash) September 8, 2022 15:45
@samuelcolvin samuelcolvin merged commit ee52496 into main Sep 8, 2022
@samuelcolvin samuelcolvin deleted the email-tweaks branch September 8, 2022 15:46
alexdrydew pushed a commit to alexdrydew/pydantic that referenced this pull request Dec 23, 2023
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting author revision deferred deferred until future release or until something else gets done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants