Re: [9front] [Bug] Mail fails to send attachments

[Romano <unobe@cpan.org>]

I don't get a suicide but noticed last night that Nail's comp.c adds content-type and content-enconding (IIRC; I'm in the desert right now). I removed those from the comp.c and added to my /mail/box/$user/headers and that resolved the issue, but there are still two content-type and content-encoding for the sent email, based on on the rawheader file. I haven't been able to test more--perhaps the marshal doc isn't correct re: adding headers from the headers file when none exist, and just always adds them.

On February 19, 2021 8:13:36 PM UTC, sirjofri <sirjofri+ml-9front@sirjofri.de> wrote:
>
>19.02.2021 20:56:00 theinicke@bss-wf.de:
>> Today I have noticed that since the rewrite Mail fails to send 
>> attachments.
>> That is if one includes an "Attach: /some/file" line and posts it,
>> (if file exists) marshal suicides like so: 66286: marshal 66286: sys:
>
>> write on closed pipe pc=0x25e04
>> leaving us with a sent mail without the attachments.
>>
>> I have not been able to track down the cause yet.
>> Have reproduced it on a clean install updated to latest HEAD
>> to make sure it is not caused by any of my modifications.
>> Here are the symptoms:
>>
>> If in /sys/src/cmd/upas/marshal/marshal.c:421 attachment function is 
>> called, then
>> in /sys/src/cmd/upas/marshal/marshal.c:430 the error occurs.
>
>Same here. The same happens when using Include: instead of Attach. Ori
>is 
>unable to reproduce this, afaik. Using marshal -8 directly works fine 
>btw.
>
>sirjofri