On Fri, 01 Feb 2019 11:36:59 +0100 Jordi Mallach <jo...@debian.org> wrote:
> El dj. 31 de 01 de 2019 a les 20:06 +0000, en/na Matthew Hall va
> escriure:
> > No problem!
> >
> > It seems to actually be a known issue(?) with SOGo upstream, I’ve
> > managed to open a bug report on there own tracker:
> >
> > https://sogo.nu/bugs/view.php?id=4659
> >
> > Looks like at least one other punter is having the same issue, which
> > I guess makes it reproducible!
> >
> > Presumably the Debian package will be rebuilt from their sources once
> > they’ve patched it?
>
> Yes, once there's a patch we would include it in our version ASAP.
>
> If you're monitoring the issue, feel free to ping us here if you see a
> fix has landed.
>
> Jordi
> --
> Jordi Mallach Pérez -- Debian developer https://www.debian.org/
> jo...@sindominio.net jo...@debian.org https://www.sindominio.net/
> GnuPG public key information available at https://oskuro.net/
>
I just posted the following on the SOGO bugtracker issue page.
Patched fixed the problem for me:
I figured out the problem. The shouldAskReceipt function in UIxMailView.m
sends a nil email parameter to NGMailAddressParser which calls
[NSString stringWithString] with this parameter. Prior versions of gnustep
tolerated this but not 1.26.
A quick fix is below:
diff -u -r b/SOGo-4.0.5/UI/MailerUI/UIxMailView.m
a/SOGo-4.0.5/UI/MailerUI/UIxMailView.m
--- b/SOGo-4.0.5/UI/MailerUI/UIxMailView.m 2019-01-09
18:03:12.000000000 +0300
+++ a/SOGo-4.0.5/UI/MailerUI/UIxMailView.m 2019-02-11
15:25:44.870164041 +0300
@@ -716,14 +716,17 @@
email = [mailHeaders objectForKey: @"return-receipt-to"];
}
- // email here can be "f...@bar.com" or "Foo Bar <f...@bar.com>"
- // we must extract the actual email address
- mailAddress = [[NGMailAddressParser mailAddressParserWithString:
email] parse];
- if ([mailAddress isKindOfClass: [NGMailAddress class]])
- email = [mailAddress address];
- else
- email = nil;
+ if (email)
+ {
+ // email here can be "f...@bar.com" or "Foo Bar <f...@bar.com>"
+ // we must extract the actual email address
+ mailAddress = [[NGMailAddressParser
mailAddressParserWithString: email] parse];
+ if ([mailAddress isKindOfClass: [NGMailAddress class]])
+ email = [mailAddress address];
+ else
+ email = nil;
+ }
if (email)
{
