GithubHelp home page GithubHelp logo

Comments (4)

jrgm avatar jrgm commented on June 11, 2024

So, one thing that is happening is (from tracing raw network traffic w/ ngrep) is that more than one email message can be delivered on a connection. So for a sender sequence like:

Connect
EHLO
MAIL FROM: msgid1
RCPT TO: mail1@
DATA
blah, blah, blah
NOOP. (gets 502 Not implemented)
MAIL FROM: msgid2
RCPT TO: mail2@
DATA
blah, blah, blah
QUIT (gets 221 Bye!)

we'll log:

[2017-06-06T14:56:37.688Z] Handling SMTP request
[2017-06-06T14:56:38.084Z] Received message for [ '[email protected]' ]
[2017-06-06T14:56:55.733Z] Received message for [ '[email protected]',
  '[email protected]' ]

Need to poke at this more, but thought I'd pass this along. And I don't know if this is the core problem reported in this issue.

from restmail.net.

jrgm avatar jrgm commented on June 11, 2024

Check out http://restmail.net/mail/[email protected] and
http://restmail.net/mail/[email protected]. (Note: there may have been other mails to these addresses not shown above, but http://restmail.net/mail/[email protected]'s mailbox has mail for [email protected] (no CC: involved)). Maybe need to handle the envelope RCPT TO, to handle these multiple payloads per connection cleanly.

from restmail.net.

jrgm avatar jrgm commented on June 11, 2024

p.s., I can switch restmail back to the previous release, if needed.

from restmail.net.

rfk avatar rfk commented on June 11, 2024

So, one thing that is happening is (from tracing raw network traffic w/ ngrep)
is that more than one email message can be delivered on a connection.

Just to confirm, I tried this out by hand and it did indeed cause restmail to deliver both emails to both addresses, rather just one to each address. So it seems plausible that this is the cause of the bug. My proposed fix is in #22 and was actually a pretty small change overall.

from restmail.net.

Related Issues (18)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.