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

Catch All parse failed #569

Closed
marcelinhov2 opened this issue Aug 28, 2020 · 8 comments · Fixed by #714
Closed

Catch All parse failed #569

marcelinhov2 opened this issue Aug 28, 2020 · 8 comments · Fixed by #714

Comments

@marcelinhov2
Copy link

Hey, I'm getting an error that I think could be avoided.

For outlook server sometimes I get this return:
image

If I'm not wrong, this is a catch all return, am I right?

Thanks.

@amaury1093
Copy link
Member

Hmm, looking at https://answers.microsoft.com/en-us/outlook_com/forum/all/server-error-452-453-error-too-many-recipients/6e741eef-c85e-4fb6-981d-cf6a65c755ab, it seems it's just a rate-limiting error. What do you think?

@slimakcz
Copy link

I have the same issue. When I trying check_if_email_exists the result is "transient: 4.5.3 Too many recipients (AS780090) [SN1NAM02FT003.eop-nam02.prod.protection.outlook.com]", but manual check from the same server is ok.
obrazek

@amaury1093
Copy link
Member

@slimakcz Interesting, could you run the same command with RUST_LOG=debug check_if_email_exists .... This will show the intermediate SMTP commands check_if_email_exists is using, and we can compare to the telnet ones.

@slimakcz
Copy link

slimakcz commented Nov 11, 2020

@amaurymartiny
[2020-11-11T11:35:24Z DEBUG async_smtp::smtp::client::inner] >> MAIL FROM:slimak@xxx
[2020-11-11T11:35:24Z DEBUG async_smtp::smtp::client::inner] << 250 2.1.0 Sender OK
[2020-11-11T11:35:24Z DEBUG async_smtp::smtp::client::inner] >> RCPT TO:z88EvVgn94m8Hcl@xxx
[2020-11-11T11:35:24Z DEBUG async_smtp::smtp::client::inner] << 250 2.1.5 Recipient OK
[2020-11-11T11:35:24Z DEBUG async_smtp::smtp::client::inner] >> RCPT TO:asilva@xxx
[2020-11-11T11:35:25Z DEBUG async_smtp::smtp::client::inner] << 452 4.5.3 Too many recipients (AS780090) [CY1NAM02FT054.eop-nam02.prod.protection.outlook.com]

the "RCPT TO:z88EvVgn94m8Hcl@xxx" looks strange

@amaury1093
Copy link
Member

amaury1093 commented Nov 11, 2020

AH. the z88 one is to check for catch-all: we generate a random username, and test {random_username}@{domain}. If it's 250, then the provider is catch-all.

I think the best would be to do:

  • if it's catch-all, then we skip checking the inputted email
  • if it's not catch-all, then we check the inputted email.

https://github.com/amaurymartiny/check-if-email-exists/blob/18a97afa08c32ca72fa9ac6476bbc93618558b5e/core/src/smtp/mod.rs#L357-L360

@slimakcz Would you like to create a PR?

@slimakcz
Copy link

@amaurymartiny thank you so much for very fast answer.
I can't program in RUST, so I need to find some colleagues to help me :-)
And I find another false negative result:
obrazek
Maybe will be better if catch-all is negative to open new connection just with inputted email and create switch if user wan't to try catch_all (for save new connections).
I must more thinking about it, I am not sure if this solutions is best, maybe have some consequences and also I don't know if is this solution good for you.

@amaury1093
Copy link
Member

@slimakcz I just released 0.8.15, which I think should solve both issues, can you confirm?

@slimakcz
Copy link

@amaurymartiny yes, both issue was solved. thank you so much

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 a pull request may close this issue.

3 participants