Hello,
I borrowed dovecot parser for email addresses and going to use it in new perl module as replacement for Email::Address. During implementation and testing I found bugs in dovecot parser & generator. So I'm sending my patches together with tests.
Pali Rohár (7): lib-mail: message_address_write: Fix generating empty group list lib-mail: message_address_write: Fix generating group list with empty name lib-mail: parse_addr_spec: Like in rfc822_skip_comment() check if last_comment is not NULL lib-mail: parse_addr_spec: Email address without local-part is invalid lib-mail: parse_mailbox: Set display name instead mailbox when parsing failed lib-mail: message_address_write: Quote and escape strings if needed lib-mail: Update tests for message address
src/lib-mail/message-address.c | 99 +++++++++++++++++++++++++++++++---- src/lib-mail/test-message-address.c | 11 +++- 2 files changed, 98 insertions(+), 12 deletions(-)
-- 1.7.9.5
On April 2, 2016 at 6:58 PM Pali Rohár pali.rohar@gmail.com wrote:
Hello,
I borrowed dovecot parser for email addresses and going to use it in new perl module as replacement for Email::Address. During implementation and testing I found bugs in dovecot parser & generator. So I'm sending my patches together with tests.
Pali Rohár (7): lib-mail: message_address_write: Fix generating empty group list lib-mail: message_address_write: Fix generating group list with empty name lib-mail: parse_addr_spec: Like in rfc822_skip_comment() check if last_comment is not NULL lib-mail: parse_addr_spec: Email address without local-part is invalid lib-mail: parse_mailbox: Set display name instead mailbox when parsing failed lib-mail: message_address_write: Quote and escape strings if needed lib-mail: Update tests for message address
src/lib-mail/message-address.c | 99 +++++++++++++++++++++++++++++++---- src/lib-mail/test-message-address.c | 11 +++- 2 files changed, 98 insertions(+), 12 deletions(-)
-- 1.7.9.5
Hi!
Did you post the patches somewhere as they do not seem to have made it thru.
Aki Tuomi Dovecot Oy
On Sunday 03 April 2016 12:45:23 aki.tuomi@dovecot.fi wrote:
On April 2, 2016 at 6:58 PM Pali Rohár pali.rohar@gmail.com wrote:
Hello,
I borrowed dovecot parser for email addresses and going to use it in new perl module as replacement for Email::Address. During implementation and testing I found bugs in dovecot parser & generator. So I'm sending my patches together with tests.
Pali Rohár (7): lib-mail: message_address_write: Fix generating empty group list lib-mail: message_address_write: Fix generating group list with empty
name
lib-mail: parse_addr_spec: Like in rfc822_skip_comment() check if
last_comment is not NULL
lib-mail: parse_addr_spec: Email address without local-part is
invalid
lib-mail: parse_mailbox: Set display name instead mailbox when
parsing failed
lib-mail: message_address_write: Quote and escape strings if needed lib-mail: Update tests for message address
src/lib-mail/message-address.c | 99
+++++++++++++++++++++++++++++++----
src/lib-mail/test-message-address.c | 11 +++- 2 files changed, 98 insertions(+), 12 deletions(-)
Hi!
Did you post the patches somewhere as they do not seem to have made it thru.
Aki Tuomi Dovecot Oy
Hi! I sent all patches to ML, but dovecot server refused them. I already contacted dovecot-owner@ about this problem, but without response yet.
-- Pali Rohár pali.rohar@gmail.com
participants (2)
-
aki.tuomi@dovecot.fi
-
Pali Rohár