* bug#45015: [mumi] 44964 cannot be displayed
@ 2020-12-02 20:26 Ricardo Wurmus
[not found] ` <handler.45015.B.16069407982867.ack@debbugs.gnu.org>
2020-12-04 6:11 ` Arun Isaac
0 siblings, 2 replies; 7+ messages in thread
From: Ricardo Wurmus @ 2020-12-02 20:26 UTC (permalink / raw)
To: 45015
Display of issue 44964 fails because the email address is not properly
split.
--8<---------------cut here---------------start------------->8---
GET /44964
In mumi/web/server.scm:
33:9 9 (handler _ _)
In mumi/web/controller.scm:
122:24 8 (_ _)
In mumi/web/view/html.scm:
358:2 7 (issue-page #<<bug-status> num: 44964 archived: #f bloâ¦> â¦)
In unknown file:
6 (filter #<procedure 7f19e5295440 at mumi/web/view/htmlâ¦> â¦)
In mumi/web/view/html.scm:
363:24 5 (_ _)
In mumi/messages.scm:
142:8 4 (internal-message? _)
In unknown file:
3 (_ #<procedure 7f174074bfe0 at ice-9/boot-9.scm:792:28â¦> â¦)
In mumi/messages.scm:
88:6 2 (sender _)
In unknown file:
1 (car "Jorge P. de Morais Neto <jorge+list@disroot.org>")
In ice-9/boot-9.scm:
1669:16 0 (raise-exception _ #:continuable? _)
--8<---------------cut here---------------end--------------->8---
--
Ricardo
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#45015: [mumi] 44964 cannot be displayed
[not found] ` <handler.45015.B.16069407982867.ack@debbugs.gnu.org>
@ 2020-12-04 5:59 ` Ricardo Wurmus
2020-12-04 19:36 ` Arun Isaac
0 siblings, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2020-12-04 5:59 UTC (permalink / raw)
To: 45015
Mumi commit 6fd49e9341d5673b70edfb333a4e5fa0a44128d2 works around this
problem, which I suspect might be a bug in guile-email.
For some reason “Jorge P. de Morais Neto <jorge+list@disroot.org>” is
not split up into name and address parts but remains a single string.
--
Ricardo
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#45015: [mumi] 44964 cannot be displayed
2020-12-02 20:26 bug#45015: [mumi] 44964 cannot be displayed Ricardo Wurmus
[not found] ` <handler.45015.B.16069407982867.ack@debbugs.gnu.org>
@ 2020-12-04 6:11 ` Arun Isaac
2020-12-04 7:27 ` Ricardo Wurmus
1 sibling, 1 reply; 7+ messages in thread
From: Arun Isaac @ 2020-12-04 6:11 UTC (permalink / raw)
To: Ricardo Wurmus, 45015
[-- Attachment #1: Type: text/plain, Size: 579 bytes --]
Hi,
This is likely a bug in guile-email. It shouldn't have returned the
unparsed From address. It was likely triggered by "Jorge P. de Morais
Neto <jorge+list@disroot.org>" being an invalid address. The period
character is not allowed in the name and should have been quoted like
"\"Jorge P. de Morais Neto\" <jorge+list@disroot.org>". I will
investigate and come up with some fix within 24 hours.
Meanwhile, do CC me or guile-email@systemreboot.net on anything
guile-email related. That would help me take note sooner. I found this
bug report almost by chance.
Thanks,
Arun
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 524 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#45015: [mumi] 44964 cannot be displayed
2020-12-04 6:11 ` Arun Isaac
@ 2020-12-04 7:27 ` Ricardo Wurmus
0 siblings, 0 replies; 7+ messages in thread
From: Ricardo Wurmus @ 2020-12-04 7:27 UTC (permalink / raw)
To: Arun Isaac; +Cc: 45015
Arun Isaac <arunisaac@systemreboot.net> writes:
> This is likely a bug in guile-email. It shouldn't have returned the
> unparsed From address. It was likely triggered by "Jorge P. de Morais
> Neto <jorge+list@disroot.org>" being an invalid address. The period
> character is not allowed in the name and should have been quoted like
> "\"Jorge P. de Morais Neto\" <jorge+list@disroot.org>". I will
> investigate and come up with some fix within 24 hours.
Thank you!
> Meanwhile, do CC me or guile-email@systemreboot.net on anything
> guile-email related. That would help me take note sooner. I found this
> bug report almost by chance.
You found it just in time. I only just put you on Cc after I became
convinced that this might be a parser failure.
--
Ricardo
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#45015: [mumi] 44964 cannot be displayed
2020-12-04 5:59 ` Ricardo Wurmus
@ 2020-12-04 19:36 ` Arun Isaac
2020-12-06 11:30 ` Ricardo Wurmus
0 siblings, 1 reply; 7+ messages in thread
From: Arun Isaac @ 2020-12-04 19:36 UTC (permalink / raw)
To: Ricardo Wurmus, 45015
[-- Attachment #1: Type: text/plain, Size: 1207 bytes --]
> For some reason “Jorge P. de Morais Neto <jorge+list@disroot.org>” is
> not split up into name and address parts but remains a single string.
I have fixed this bug in the latest guile-email master, and added
tests to prevent regression. See
https://git.systemreboot.net/guile-email/commit/?id=37e245539bc9e27db35731e7b517cdde397834e2
and
https://git.systemreboot.net/guile-email/commit/?id=907a5d678c58eff653133df441aeecd3b7025e37
I thought “Jorge P. de Morais Neto <jorge+list@disroot.org>” was an
invalid email address due to the period character, but apparently it is
supported by the obsolete addressing specification of RFC5322 (see
https://tools.ietf.org/html/rfc5322#section-4). I hadn't implemented the
obsolete parts of the specification until now.
> Mumi commit 6fd49e9341d5673b70edfb333a4e5fa0a44128d2 works around this
> problem, which I suspect might be a bug in guile-email.
guile-email should guarantee the parsed From header is an alist of the
appropriate form. So, I vote for reverting this commit, thus reducing
complexity on mumi's end. But, I understand that guile-email is not yet
doing a great job of providing these strong guarantees.
Cheers!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 524 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#45015: [mumi] 44964 cannot be displayed
2020-12-04 19:36 ` Arun Isaac
@ 2020-12-06 11:30 ` Ricardo Wurmus
2020-12-06 11:50 ` Arun Isaac
0 siblings, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2020-12-06 11:30 UTC (permalink / raw)
To: Arun Isaac; +Cc: 45015
Arun Isaac <arunisaac@systemreboot.net> writes:
>> For some reason “Jorge P. de Morais Neto <jorge+list@disroot.org>” is
>> not split up into name and address parts but remains a single string.
>
> I have fixed this bug in the latest guile-email master, and added
> tests to prevent regression. See
> https://git.systemreboot.net/guile-email/commit/?id=37e245539bc9e27db35731e7b517cdde397834e2
> and
> https://git.systemreboot.net/guile-email/commit/?id=907a5d678c58eff653133df441aeecd3b7025e37
Thank you! You are always very fast and thorough; I appreciate this a lot!
>> Mumi commit 6fd49e9341d5673b70edfb333a4e5fa0a44128d2 works around this
>> problem, which I suspect might be a bug in guile-email.
>
> guile-email should guarantee the parsed From header is an alist of the
> appropriate form. So, I vote for reverting this commit, thus reducing
> complexity on mumi's end.
I agree. I wasn’t sure how difficult it would be to fix it in
guile-email, so I added the workaround just in case it takes longer.
I’ll upgrade and revert that commit soon.
--
Ricardo
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#45015: [mumi] 44964 cannot be displayed
2020-12-06 11:30 ` Ricardo Wurmus
@ 2020-12-06 11:50 ` Arun Isaac
0 siblings, 0 replies; 7+ messages in thread
From: Arun Isaac @ 2020-12-06 11:50 UTC (permalink / raw)
To: Ricardo Wurmus; +Cc: 45015
[-- Attachment #1: Type: text/plain, Size: 444 bytes --]
>> I have fixed this bug in the latest guile-email master, and added
>> tests to prevent regression. See
>> https://git.systemreboot.net/guile-email/commit/?id=37e245539bc9e27db35731e7b517cdde397834e2
>> and
>> https://git.systemreboot.net/guile-email/commit/?id=907a5d678c58eff653133df441aeecd3b7025e37
>
> Thank you! You are always very fast and thorough; I appreciate this a
> lot!
Thank you, I really appreciate the acknowledgement! :-)
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 524 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-12-06 11:54 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-02 20:26 bug#45015: [mumi] 44964 cannot be displayed Ricardo Wurmus
[not found] ` <handler.45015.B.16069407982867.ack@debbugs.gnu.org>
2020-12-04 5:59 ` Ricardo Wurmus
2020-12-04 19:36 ` Arun Isaac
2020-12-06 11:30 ` Ricardo Wurmus
2020-12-06 11:50 ` Arun Isaac
2020-12-04 6:11 ` Arun Isaac
2020-12-04 7:27 ` Ricardo Wurmus
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.