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

Incorrect DEBUG logging for event 'OnDeliverMessage' #181

jimimaseye opened this issue Jun 5, 2016 · 1 comment

Incorrect DEBUG logging for event 'OnDeliverMessage' #181

jimimaseye opened this issue Jun 5, 2016 · 1 comment


Copy link

@jimimaseye jimimaseye commented Jun 5, 2016

Sub OnDeliverMessage(oMessage)
logs as
"DEBUG" "Executing event OnMessageDeliver" - they dont match (wrong way round)

(where as others eg, OnDeliveryStart(oMessage), are reflected in debug correct "DEBUG" "Executing event OnDeliveryStart" - they match)

Could cause some confusion for people causing doubt as to what to believe (as I did today).

(Admittedly as priorities go this is lower than the lowest even if there is nothing else to do, but even so we are all for perfection. Right? ;-) )

Copy link

@martinknafve martinknafve commented Jun 13, 2016

Fixed for 5.7.

RvdHout added a commit to RvdHout/hmailserver that referenced this issue Aug 6, 2017
- 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
- (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
None yet
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants