Skip to content

Conversation

mverbert
Copy link

Hi,
here are 3 trivial build fixes / improvements.

Matthieu

pjstevns and others added 30 commits March 16, 2019 19:16
adding travis label
Inadvertently changed in:
4de4bc5 ("New packaging with systemd", 2017-12-20)
Set client_state CLIENT_ERR
if connection was closed from client side
as respnse like "... OK [MODIFIED 7,9] ..." not "...OK [MODIFIED [7,9]]..."
Reinstate systemd notify service type
fix detection of systemd libs on systemd >= 230
According to RFC7162 section 3.1.3.0 MODIFIED keyword should be used
cncioranu and others added 30 commits April 2, 2020 10:09
When the content-type header is not present in an email, dbmail generates a possibly incomplete BODYSTRUCTURE:
BODYSTRUCTURE ("text" "plain" NIL NIL "7BIT" 4669 2294 NIL NIL NIL NIL)
Rather than:
BODYSTRUCTURE ("text" "plain" NIL NIL NIL "7BIT" 4669 2294 NIL NIL NIL NIL)

Non java clients seem to be fine with this, but java clients barf, sometimes quietly.

Looking through the code, I believe this difference is unintended.

It seems to be due to a call in dm_misc.c to:
imap_append_hash_as_string(GList *list, const char *type)
Where type is null, which happens when a call to:
g_mime_object_get_header(object, "Content-Type")
returns a null for the Content-Type, ie, when there is no Content-Type header.

The code in imap_append_hash_as_string returns an unmodified list when the passed-in type is null, where I believe it should append a 'NIL' and return the list.

I will submit a merge request with this implemented shortly, and the change does fix the issue in my environment.

*nbeta-git
added trace (according to review)
…lid_bodystructure

merge from awheeler:fix_imap_invalid_bodystructure
Simplify and fix AC_CHECK_HEADERS for libzdb
works with libzdb +3.2.2
update from master in order to recompile config.
…en dburi was used to configure database connection.
…_addition

dburi application-name addition
Reconfigured all configuration files
libzdb has been skipping with a .pc file for years, and using it simplifies
locating the headers and library.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants