meli
/
meli
9
Fork 13
Code Issues 13 Pull Requests 11 Actions Packages Projects 1 Releases 19 Wiki Activity

SMTP mail_transaction triggers odd issue with greenmail #307

New Issue
Closed
opened 2023-10-02 22:30:09 +03:00 by matzipan · 0 comments
matzipan commented 2023-10-02 22:30:09 +03:00
Owner

melib version 0.8.2

Code:

extern crate melib;

use melib::futures;
use melib::smol;
use melib::smtp::*;

fn main() {
    let conf = SmtpServerConf {
        hostname: "localhost".into(),
        port: 3025,
        security: SmtpSecurity::None,
        extensions: SmtpExtensionSupport::default(),
        auth: SmtpAuth::None,
        envelope_from: "l123@mail.gr".into(),
    };

    std::thread::Builder::new()
        .spawn(move || {
            let ex = smol::Executor::new();
            futures::executor::block_on(ex.run(futures::future::pending::<()>()));
        })
        .unwrap();

    let mut conn = futures::executor::block_on(SmtpConnection::new_connection(conf)).unwrap();
    futures::executor::block_on(conn.mail_transaction(
        r#"To: l10@mail.gr
Subject: Fwd: SMTP TEST
From: Me <l15@mail.gr>
Message-Id: <E1hSjnr-0003fN-RL@pppppp>
Date: Mon, 13 Jul 2020 09:02:15 +0300

Prescriptions-R-X"#,
        Some(&[melib::Address::new(None, "l15@mail.gr".into())]),
    ))
    .unwrap();
}

Run greenmail:

docker pull greenmail/standalone:2.0.0
docker run -it --rm -e GREENMAIL_OPTS='-Dgreenmail.setup.test.all -Dgreenmail.hostname=0.0.0.0 -Dgreenmail.auth.disabled -Dgreenmail.verbose' -p 3025:3025 -p 3110:3110 -p 3143:3143 -p 3465:3465 -p 3993:3993 -p 3995:3995 -p 8080:8080 greenmail/standalone:2.0.0

The following log will be produced:

2023-10-02 18:49:10,999 DEBUG    util.LineLoggingBuffer| S: 220 /10.0.2.100 GreenMail SMTP Service v2.0.0 ready\r\n
2023-10-02 18:49:10,999 DEBUG    util.LineLoggingBuffer| C: EHLO meli.delivery\r\n
2023-10-02 18:49:10,999 DEBUG    util.LineLoggingBuffer| S: 250-/10.0.2.100\r\n
2023-10-02 18:49:11,000 DEBUG    util.LineLoggingBuffer| S: 250 AUTH PLAIN LOGIN\r\n
2023-10-02 18:49:11,081 DEBUG    util.LineLoggingBuffer| C: MAIL FROM:<l123@mail.gr>\r\n
2023-10-02 18:49:11,082 DEBUG    util.LineLoggingBuffer| S: 250 OK\r\n
2023-10-02 18:49:11,163 DEBUG    util.LineLoggingBuffer| C: RCPT TO:<l15@mail.gr>\r\n
2023-10-02 18:49:11,164 DEBUG    util.LineLoggingBuffer| S: 250 OK\r\n
2023-10-02 18:49:11,246 DEBUG    util.LineLoggingBuffer| C: DATA\r\n
2023-10-02 18:49:11,246 DEBUG    util.LineLoggingBuffer| S: 354 Start mail input; end with <CRLF>.<CRLF>\r\n
2023-10-02 18:49:11,327 DEBUG    util.LineLoggingBuffer| C: To: l10@mail.gr\r\n
2023-10-02 18:49:11,327 DEBUG    util.LineLoggingBuffer| C: Subject: Fwd: SMTP TEST\r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: From: Me <l15@mail.gr>\r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: Message-Id: <E1hSjnr-0003fN-RL@pppppp>\r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: Date: Mon, 13 Jul 2020 09:02:15 +0300\r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: \r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: Prescriptions-R-X\r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: .\r\n
2023-10-02 18:49:11,328 DEBUG    util.LineLoggingBuffer| C: .\r\n
2023-10-02 18:49:11,329 DEBUG    util.LineLoggingBuffer| S: 250 OK\r\n
2023-10-02 18:49:11,329 DEBUG    util.LineLoggingBuffer| S: 500 Invalid command. Must be 4 characters\r\n

If I use python's smtplib I get the following log:

2023-10-02 19:07:22,897 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 220 /10.0.2.100 GreenMail SMTP Service v1.6.8 ready\r\n
2023-10-02 19:07:32,103 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: ehlo [192.168.178.53]\r\n
2023-10-02 19:07:32,103 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 250-/10.0.2.100\r\n
2023-10-02 19:07:32,104 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 250 AUTH PLAIN LOGIN\r\n
2023-10-02 19:07:32,104 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: mail FROM:<bla@bla.com>\r\n
2023-10-02 19:07:32,115 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 250 OK\r\n
2023-10-02 19:07:32,155 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: rcpt TO:<bla@bla.com>\r\n
2023-10-02 19:07:32,156 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 250 OK\r\n
2023-10-02 19:07:32,196 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: data\r\n
2023-10-02 19:07:32,197 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 354 Start mail input; end with <CRLF>.<CRLF>\r\n
2023-10-02 19:07:32,238 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: Subject: Fwd: SMTP TEST\r\n
2023-10-02 19:07:32,238 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: From: Me <l15@mail.gr>\r\n
2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: Message-Id: <E1hSjnr-0003fN-RL@pppppp>\r\n
2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: Date: Mon, 13 Jul 2020 09:02:15 +0300\r\n
2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: Prescriptions-R-X\r\n
2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| C: .\r\n
2023-10-02 19:07:32,275 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG    util.LineLoggingBuffer| S: 250 OK\r\n

The reason for this error message is the extra .\r\n at the end. Removing it gets rid of the error.

melib version 0.8.2 Code: ``` extern crate melib; use melib::futures; use melib::smol; use melib::smtp::*; fn main() { let conf = SmtpServerConf { hostname: "localhost".into(), port: 3025, security: SmtpSecurity::None, extensions: SmtpExtensionSupport::default(), auth: SmtpAuth::None, envelope_from: "l123@mail.gr".into(), }; std::thread::Builder::new() .spawn(move || { let ex = smol::Executor::new(); futures::executor::block_on(ex.run(futures::future::pending::<()>())); }) .unwrap(); let mut conn = futures::executor::block_on(SmtpConnection::new_connection(conf)).unwrap(); futures::executor::block_on(conn.mail_transaction( r#"To: l10@mail.gr Subject: Fwd: SMTP TEST From: Me <l15@mail.gr> Message-Id: <E1hSjnr-0003fN-RL@pppppp> Date: Mon, 13 Jul 2020 09:02:15 +0300 Prescriptions-R-X"#, Some(&[melib::Address::new(None, "l15@mail.gr".into())]), )) .unwrap(); } ``` Run greenmail: ``` docker pull greenmail/standalone:2.0.0 docker run -it --rm -e GREENMAIL_OPTS='-Dgreenmail.setup.test.all -Dgreenmail.hostname=0.0.0.0 -Dgreenmail.auth.disabled -Dgreenmail.verbose' -p 3025:3025 -p 3110:3110 -p 3143:3143 -p 3465:3465 -p 3993:3993 -p 3995:3995 -p 8080:8080 greenmail/standalone:2.0.0 ``` The following log will be produced: ``` 2023-10-02 18:49:10,999 DEBUG util.LineLoggingBuffer| S: 220 /10.0.2.100 GreenMail SMTP Service v2.0.0 ready\r\n 2023-10-02 18:49:10,999 DEBUG util.LineLoggingBuffer| C: EHLO meli.delivery\r\n 2023-10-02 18:49:10,999 DEBUG util.LineLoggingBuffer| S: 250-/10.0.2.100\r\n 2023-10-02 18:49:11,000 DEBUG util.LineLoggingBuffer| S: 250 AUTH PLAIN LOGIN\r\n 2023-10-02 18:49:11,081 DEBUG util.LineLoggingBuffer| C: MAIL FROM:<l123@mail.gr>\r\n 2023-10-02 18:49:11,082 DEBUG util.LineLoggingBuffer| S: 250 OK\r\n 2023-10-02 18:49:11,163 DEBUG util.LineLoggingBuffer| C: RCPT TO:<l15@mail.gr>\r\n 2023-10-02 18:49:11,164 DEBUG util.LineLoggingBuffer| S: 250 OK\r\n 2023-10-02 18:49:11,246 DEBUG util.LineLoggingBuffer| C: DATA\r\n 2023-10-02 18:49:11,246 DEBUG util.LineLoggingBuffer| S: 354 Start mail input; end with <CRLF>.<CRLF>\r\n 2023-10-02 18:49:11,327 DEBUG util.LineLoggingBuffer| C: To: l10@mail.gr\r\n 2023-10-02 18:49:11,327 DEBUG util.LineLoggingBuffer| C: Subject: Fwd: SMTP TEST\r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: From: Me <l15@mail.gr>\r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: Message-Id: <E1hSjnr-0003fN-RL@pppppp>\r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: Date: Mon, 13 Jul 2020 09:02:15 +0300\r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: \r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: Prescriptions-R-X\r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: .\r\n 2023-10-02 18:49:11,328 DEBUG util.LineLoggingBuffer| C: .\r\n 2023-10-02 18:49:11,329 DEBUG util.LineLoggingBuffer| S: 250 OK\r\n 2023-10-02 18:49:11,329 DEBUG util.LineLoggingBuffer| S: 500 Invalid command. Must be 4 characters\r\n ``` If I use python's smtplib I get the following log: ``` 2023-10-02 19:07:22,897 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 220 /10.0.2.100 GreenMail SMTP Service v1.6.8 ready\r\n 2023-10-02 19:07:32,103 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: ehlo [192.168.178.53]\r\n 2023-10-02 19:07:32,103 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 250-/10.0.2.100\r\n 2023-10-02 19:07:32,104 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 250 AUTH PLAIN LOGIN\r\n 2023-10-02 19:07:32,104 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: mail FROM:<bla@bla.com>\r\n 2023-10-02 19:07:32,115 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 250 OK\r\n 2023-10-02 19:07:32,155 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: rcpt TO:<bla@bla.com>\r\n 2023-10-02 19:07:32,156 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 250 OK\r\n 2023-10-02 19:07:32,196 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: data\r\n 2023-10-02 19:07:32,197 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 354 Start mail input; end with <CRLF>.<CRLF>\r\n 2023-10-02 19:07:32,238 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: Subject: Fwd: SMTP TEST\r\n 2023-10-02 19:07:32,238 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: From: Me <l15@mail.gr>\r\n 2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: Message-Id: <E1hSjnr-0003fN-RL@pppppp>\r\n 2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: Date: Mon, 13 Jul 2020 09:02:15 +0300\r\n 2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: Prescriptions-R-X\r\n 2023-10-02 19:07:32,239 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| C: .\r\n 2023-10-02 19:07:32,275 smtp:0.0.0.0:3025<-/10.0.2.100:51902 DEBUG util.LineLoggingBuffer| S: 250 OK\r\n ``` The reason for this error message is the extra .\r\n at the end. Removing it gets rid of the error.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
wip
master
starlark
jump_to
notmuch-fcc-dirs
escape8259-vendor
fix/gitea-ci-artifacts
feature/sieve
feature/async-job-delay-wait
fix/269-invalid-ctext-loop
imap-codec-server-tests
async-cursors
feature/export-vcards
imap-connection-changes
feature/search-all-mailboxes
sieve
feature/readline-command-parsing
feature/perform-shortcut
pipe_to_clipboard
jmap-status-and-connect-retry-wip
issue-133
mutt2meli
imap-lemonade
lazy_fetch
jmap-eventsource
memfd
jmap
embed
v0.8.5
v0.8.5-rc.3
v0.8.5-rc.2
v0.8.5-rc.1
v0.8.4
v0.8.3
v0.8.2
v0.8.1
v0.8.0
v0.1.1
v0.1.0
alpha-0.7.2
alpha-0.7.1
alpha-0.7.0
alpha-0.6.2
alpha-0.6.1
alpha-0.6.0
alpha-0.5.1
alpha-0.5.0
alpha-0.4.2
alpha-0.4.1
pre-alpha-0.4.0
pre-alpha-0.3.2
pre-alpha-0.3.1
pre-alpha-0.3.0
pre-alpha-0.2.0
Labels
Clear labels   
IMAP

Related to the IMAP backend   
JMAP

Related to the JMAP backend   
Maildir

Related to the Maildir backend   
Retired

Issues that have been open so long we might as well drop them   
User Experience

Related to use and workflow with meli   
User Interface

Related to the terminal user interface   
bsd

*BSD target   
bug

Something is not working   
contacts
   
currently worked on

Stuff I am working on right now   
documentation
   
duplicate

This issue or pull request already exists   
easy

Non-complex issues   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
linux-gnu

target   
macos

apple-darwin target   
mbox

Related to the mbox backend   
notmuch

Related to the notmuch backend   
question

More information is needed   
security
   
wishlist

New features that would be cool to have, but are not priority   
wontfix

This won't be fixed
No Label
IMAP
JMAP
Maildir
Retired
User Experience
User Interface
bsd
bug
contacts
currently worked on
documentation
duplicate
easy
enhancement
help wanted
invalid
linux-gnu
macos
mbox
notmuch
question
security
wishlist
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
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: meli/meli#307
There is no content yet.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?