starred/cypht

Fork 0

mirror of https://github.com/cypht-org/cypht.git synced 2026-04-26 13:36:02 +03:00

[GH-ISSUE #420] Sometimes, I click reply, or reply to all, and the email is blank. If I cancel and try again, it works OK. #347

New issue

Closed

opened 2026-02-25 21:34:48 +03:00 by kerem · 12 comments

kerem commented

2026-02-25 21:34:48 +03:00

Owner

Originally created by @marclaporte on GitHub (Aug 11, 2020).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/420

Originally assigned to: @jasonmunro on GitHub.

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

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

Originally created by @marclaporte on GitHub (Aug 11, 2020). Original GitHub issue: https://github.com/cypht-org/cypht/issues/420 Originally assigned to: @jasonmunro on GitHub. Seen by at least Jason and myself. No pattern to reproduce yet. https://gitter.im/cypht-org/community?at=5f307abefe39ca5d65902d7c

kerem

2026-02-25 21:34:48 +03:00

closed this issue
added the
bug

smtp
labels

kerem commented

2026-02-25 21:34:48 +03:00

Author

Owner

@marclaporte commented on GitHub (Sep 17, 2020):

@jasonmunro Can you think of anything I could do to detect the pattern?

@marclaporte commented on GitHub (Sep 17, 2020): @jasonmunro Can you think of anything I could do to detect the pattern?

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Oct 2, 2020):

I have been to reproduce something odd: forwarded message as attachment appears more than once:

To do this

On a message, click "forward", then back (with your browser), then forward again, etc.

@marclaporte commented on GitHub (Oct 2, 2020): I have been to reproduce something odd: forwarded message as attachment appears more than once: ![Screenshot_20201002-134717](https://user-images.githubusercontent.com/1004261/94954081-8f51e500-04b6-11eb-9cc7-2d6edbac5010.png) To do this * On a message, click "forward", then back (with your browser), then forward again, etc.

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@jasonmunro commented on GitHub (Feb 9, 2021):

This issue should be resolved in the master branch

@jasonmunro commented on GitHub (Feb 9, 2021): This issue should be resolved in the master branch

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Feb 9, 2021):

@jasonmunro: I still get many attachments when doing:

"On a message, click "forward", then back (with your browser), then forward again, etc."

@marclaporte commented on GitHub (Feb 9, 2021): @jasonmunro: I still get many attachments when doing: "On a message, click "forward", then back (with your browser), then forward again, etc."

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@jasonmunro commented on GitHub (Feb 9, 2021):

that should be it's own issue as it is unrelated to this particular problem

@jasonmunro commented on GitHub (Feb 9, 2021): that should be it's own issue as it is unrelated to this particular problem

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Feb 10, 2021):

I confirm blank emails are solved by github.com/jasonmunro/cypht@2a9060a17a

Thanks!

@marclaporte commented on GitHub (Feb 10, 2021): I confirm blank emails are solved by https://github.com/jasonmunro/cypht/commit/2a9060a17acf6b19abdd8f0a4e9dc84a3fb42ef7 Thanks!

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Feb 12, 2021):

I am still getting some. But it happens less often.

I also noticed some on "forward"

@marclaporte commented on GitHub (Feb 12, 2021): I am still getting some. But it happens less often. I also noticed some on "forward"

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Feb 16, 2021):

The trick I described above (On a message, click "forward", then back (with your browser), then forward again, etc.) no longer works because that issue has been fixed by https://github.com/jasonmunro/cypht/pull/453

blank message on reply sometimes still occur. See: github.com/jasonmunro/cypht@2a9060a17a (commitcomment-47201666)

@marclaporte commented on GitHub (Feb 16, 2021): The trick I described above (On a message, click "forward", then back (with your browser), then forward again, etc.) no longer works because that issue has been fixed by https://github.com/jasonmunro/cypht/pull/453 blank message on reply sometimes still occur. See: https://github.com/jasonmunro/cypht/commit/2a9060a17acf6b19abdd8f0a4e9dc84a3fb42ef7#commitcomment-47201666

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Sep 1, 2021):

This problem persists to this day, but I hope it is the same root cause as discussed at https://github.com/jasonmunro/cypht/pull/508 and thus, be resolved soon.

@marclaporte commented on GitHub (Sep 1, 2021): This problem persists to this day, but I hope it is the same root cause as discussed at https://github.com/jasonmunro/cypht/pull/508 and thus, be resolved soon.

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Oct 8, 2021):

AFAICT, this is solved with latest code. If I see the issue again, I will re-open.

@marclaporte commented on GitHub (Oct 8, 2021): AFAICT, this is solved with latest code. If I see the issue again, I will re-open.

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Mar 13, 2023):

So I was still having this intermittent problem, so @kroky dove in.

It turns out to be a race condition in AJAX requests executed in the background when Cypht is integrated in Tiki, and when "Session storage location" is DB. Switching to "Default (from php.ini) solves it. We'll experiment a bit more and add a fix to Tiki.

@marclaporte commented on GitHub (Mar 13, 2023): So I was still having this intermittent problem, so @kroky dove in. It turns out to be a race condition in AJAX requests executed in the background when Cypht is integrated in Tiki, and when "Session storage location" is DB. Switching to "Default (from php.ini) solves it. We'll experiment a bit more and add a fix to Tiki.

kerem commented

2026-02-25 21:34:49 +03:00

Author

Owner

@marclaporte commented on GitHub (Mar 13, 2023):

For the record: gitlab.com/tikiwiki/tiki/-@5281f16493

@marclaporte commented on GitHub (Mar 13, 2023): For the record: https://gitlab.com/tikiwiki/tiki/-/commit/5281f16493abe46df57678ec329f8b6124d8aa8e

kerem referenced this issue

2026-02-25 21:36:14 +03:00

[PR #347] [MERGED] use separate constant for VENDOR_PATH #789