karolyi/py3-validate-email
3
2
Fork 0
Code Issues 1 Pull requests Projects Releases Wiki Activity

Next step towards 1.0.0 #63

Merged
reinhard-mueller merged 6 commits from master into version-1.0.0 2021-03-14 13:54:32 +01:00
Conversation 2 Commits 6 Files changed 12 +344 -133
reinhard-mueller commented 2021-03-12 01:13:23 +01:00 (Migrated from github.com)
Copy link

Comments welcome!

Comments welcome!
reinhard-mueller commented 2021-03-12 08:58:24 +01:00 (Migrated from github.com)
Copy link

BTW, I will add more unittests for smtp_check to this PR soon, but you're still welcome to comment on what's already here.

BTW, I will add more unittests for `smtp_check` to this PR soon, but you're still welcome to comment on what's already here.
karolyi (Migrated from github.com) reviewed 2021-03-13 13:47:20 +01:00
FAQ.md
@ -14,3 +21,4 @@
we talk to seemingly accepts the email address.
## Everything gets rejected:
karolyi (Migrated from github.com) commented 2021-03-13 13:47:20 +01:00
Copy link

Is this really the case with MS servers? So far I've seen outlook servers (if you meant those) rejecting the invalid emails properly.

Is this really the case with MS servers? So far I've seen outlook servers (if you meant those) rejecting the invalid emails properly.
karolyi (Migrated from github.com) reviewed 2021-03-13 14:01:53 +01:00
FAQ.md
karolyi (Migrated from github.com) commented 2021-03-13 14:01:53 +01:00
Copy link

"dynamic range" is called an IP pool

"dynamic range" is called an IP pool
karolyi (Migrated from github.com) reviewed 2021-03-13 14:03:00 +01:00
FAQ.md
karolyi (Migrated from github.com) commented 2021-03-13 14:03:00 +01:00
Copy link

... servers for many email domains (depending on their configuration) will reject connections ...

... servers for many email domains (depending on their configuration) will reject connections ...
karolyi (Migrated from github.com) reviewed 2021-03-13 14:12:29 +01:00
validate_email/validate_email.py
karolyi (Migrated from github.com) commented 2021-03-13 14:12:29 +01:00
Copy link

if not check_dns and not check_smtp: is preferred, as it's more pythonic :)

`if not check_dns and not check_smtp:` is preferred, as it's more pythonic :)
karolyi (Migrated from github.com) reviewed 2021-03-13 15:05:09 +01:00
validate_email/smtp_check.py
karolyi (Migrated from github.com) commented 2021-03-13 15:05:08 +01:00
Copy link

This can look like

def smtp_check(
    email_address: EmailAddress, mx_records: List[str],
    timeout: float = 10, helo_host: Optional[str] = None,
    from_address: Optional[EmailAddress] = None, debug: bool = False
) -> bool:

(autopep8 make these changes)

This can look like ```python def smtp_check( email_address: EmailAddress, mx_records: List[str], timeout: float = 10, helo_host: Optional[str] = None, from_address: Optional[EmailAddress] = None, debug: bool = False ) -> bool: ``` (autopep8 make these changes)
karolyi (Migrated from github.com) reviewed 2021-03-13 15:22:18 +01:00
karolyi (Migrated from github.com) left a comment
Copy link

There was a couple comments I made, but aside from those, I'm very satisfied with what you did here.

You did a nice documentation job there, clever job done with the tests. Let's iron out the changes I mentioned here and the dependency questions, and then I think we're ready for the major release.

There was a couple comments I made, but aside from those, I'm very satisfied with what you did here. You did a nice documentation job there, clever job done with the tests. Let's iron out the changes I mentioned here and the dependency questions, and then I think we're ready for the major release.
reinhard-mueller (Migrated from github.com) reviewed 2021-03-14 13:17:43 +01:00
FAQ.md
@ -14,3 +21,4 @@
we talk to seemingly accepts the email address.
## Everything gets rejected:
reinhard-mueller (Migrated from github.com) commented 2021-03-14 13:17:43 +01:00
Copy link

I have tested various domains where the MX is .mail.protection.outlook.com; some of them reject invalid emails properly and some don't. If you want I can send you some examples in private email, you can find my email address [here[(https://wiki.fsfe.org/Supporters/reinhard).

I have tested various domains where the MX is <domainname>.mail.protection.outlook.com; some of them reject invalid emails properly and some don't. If you want I can send you some examples in private email, you can find my email address [here[(https://wiki.fsfe.org/Supporters/reinhard).
karolyi commented 2021-03-14 13:55:31 +01:00 (Migrated from github.com)
Copy link

This was a great contribution, thank you!

I'll change the idna dependency and make a 1.0.0 release soon.

This was a great contribution, thank you! I'll change the idna dependency and make a 1.0.0 release soon.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
stale

   
wontfix

This will not be worked on

No labels
bug
duplicate
enhancement
good first issue
help wanted
invalid
question
stale
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: karolyi/py3-validate-email#63
No description provided.
Delete branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?