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

Faulty: SMTP 'Disconnect client after too many invalid commands' #160

Closed
jimimaseye opened this Issue Apr 17, 2016 · 0 comments

Comments

Projects
None yet
2 participants
@jimimaseye
Copy link

jimimaseye commented Apr 17, 2016

Documentation at https://www.hmailserver.com/documentation/latest/?page=reference_protocolsmtp says:

Disconnect client after too many invalid commands
Using this setting you can disconnect clients which sends to many invalid commands. For example, some spammers try to send email to a lot of different addresses on your server, hoping that your server will accept at least one of them. Using this option, you can automatically disconnect clients that tries to do this.

However, a test has proven that this doesnt work as expected/written:

a, The count of invalid commands stated in the settings always need +2 more before actually being disconnected
b, An attempt to send in to an unknown address to the server does not increment/count as an invalid command at all.

Full test, explanation, and results here: https://www.hmailserver.com/forum/viewtopic.php?p=185312#p185312

@martinknafve martinknafve added this to the 5.7 milestone Apr 22, 2016

@martinknafve martinknafve added the bug label Apr 23, 2016

martinknafve added a commit that referenced this issue Apr 24, 2016

The "Disconnect client after too many invalid commands"-feature is of…
…f by 1 and does not take failing RCPT TO's into account. #160

RvdHout added a commit to RvdHout/hmailserver that referenced this issue Aug 6, 2017

- Sub OnHELO(oClient) hmailserver#153 hmailserver#153
- Fixed Incorrect DEBUG logging for event 'OnDeliverMessage' hmailserver#181 hmailserver#181
- Include HTMLBody into IMAP TEXT search hmailserver#193
- Fixed implicit conversion: "int" to "unsigned char" hmailserver#204
- Faulty: SMTP 'Disconnect client after too many invalid commands' hmailserver#160 hmailserver#160
- SMTP server error "550 Unsupported ESMTP extension" on MAIL FROM:... AUTH=<> [with fix] hmailserver#164 hmailserver#164
- Remove 15GB backup limit hmailserver#69 hmailserver#69
- Speed up 'update hm_messages set messageflags' hmailserver#221 hmailserver#221
- https://www.hmailserver.com/forum/viewtopic.php?f=7&p=196733#p196733 (Ofice 2016 Read-Receipt Bug)
- Add Return-Path as topmost header before sending the message to SA (+ delete Return-Path header after the SA check completes) hmailserver#116
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment