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

Include HTMLBody into IMAP TEXT search. #193

merged 1 commit into from Feb 19, 2017


Copy link

@M0ns1gn0r M0ns1gn0r commented Aug 4, 2016

In contrast to the SEARCH BODY, the SEARCH TEXT command didn't perform search in HTML bodies of messages . One had to resort to SEARCH OR SUBJECT "abc" BODY "abc".

Currently TEXT doesn't work as expected for messages with HTML bodies. One had to resort to `SEARCH OR SUBJECT "abc" BODY "abc"`.
Copy link

@ericnewton76 ericnewton76 commented Aug 8, 2016

Wow, does hmailserver not have a better way to search, besides loading every single message, one by one, and searching within the message body? It's effectively performing an expensive table scan, and even worse, loading up large amounts of data.

Seems like the search should get deferred into the underlying data store.

@martinknafve martinknafve merged commit 05b74ab into hmailserver:master Feb 19, 2017
RvdHout added a commit to RvdHout/hmailserver that referenced this pull request 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 issues

Successfully merging this pull request may close these issues.

None yet

3 participants