Sometimes, I attach a file to an email, and it's confirmed uploaded. But then, when I send, the attachment is not part of sent email

[marclaporte]

Seen by at least Jason and myself. No pattern to reproduce yet.

https://gitter.im/cypht-org/community?at=5f307abefe39ca5d65902d7c

[marclaporte]

But I am 99% sure root cause is identical to this other bug we can replicate: https://github.com/jasonmunro/cypht/issues/420

[marclaporte]

Ok, looks like I was wrong in previous message (!) But I am now 99% sure [ will I ever learn? :-) ] that it's related to https://github.com/jasonmunro/cypht/issues/436

To see some weirdness with attachments: On a message, click "forward", then back (with your browser), then forward again, etc. It is possible that on your first "forward" click, the email will be blank, and you'll have an attachment (mail.mime message/rfc822), then, on each browser "back" and Cypht "forward", you may end up with an extra attachment.

[marclaporte]

I got this again with latest Tiki 21.x which includes https://github.com/jasonmunro/cypht/pull/453

But I can't find a pattern to reproduce.

[marclaporte]

Related commit: https://gitlab.com/tikiwiki/tiki/-/commit/caf8c793877b6831278c59003ece3250fcd2992e

This likely solves the issue for Cypht in Tiki. But Jason had noticed this (or a similar issue) in standalone Cypht. So there is perhaps another issue, but that we can't reproduce.

So I will close this for now, and re-open if I see the behavior again.

[marclaporte]

Victor found the issue. Merge request coming soon.

[marclaporte]

Here is, with a good discussion: https://github.com/jasonmunro/cypht/pull/508

[marclaporte]

@henrique-borba: With latest code, these are two issues:

How to reproduce: 1- Click to compose a new email 2- Fill out text fields 3- Click Save (it will confirm it saved a drat) 4- Upload an image 5- Send

Issues: 1- Email is sent without attachment 2- Draft email subsists in draft folder

Thanks!

[marclaporte]

This will be solved by https://github.com/jasonmunro/cypht/pull/524

[marclaporte]

It also needed https://github.com/jasonmunro/cypht/pull/548

[marclaporte]

@NasserNgandu found a pattern which related to folder permissions where files are temporarily stored. He will add an error message if folder is not writable.

[marclaporte]

Possible fix: https://github.com/jasonmunro/cypht/pull/575

[marclaporte]

I haven't seen this issue in a while, so presuming fixed. Will re-open if I encounter again