-
Notifications
You must be signed in to change notification settings - Fork 260
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error while loading messages again #53
Comments
So some update... I just discovered that its some specific messages that cause the trouble. When the messages are not on the first page the folder loads fine. However as soon as I scroll down far enough the additional messages are not loaded. So I created a test folder with thunderbird and moved some messages to this folder. By doing so I found one malicious message that stops the folder from loading. It's a mail with several umlauts and é and an attachment. Also it is forwarded and sent through a mailinglist... so all possible reasons to fail. If I forward the mails to myself the problem is gone! What should I do next? I could send someone the problematic email but I don't want it posted here! Any help is appreciated |
|
Got this too, happens when you receive an email with a hidden recipient list. The following fix works: diff --git a/lib/Model/ConvertAddresses.php b/lib/Model/ConvertAddresses.php
index f723826c..78f5c554 100644
--- a/lib/Model/ConvertAddresses.php
+++ b/lib/Model/ConvertAddresses.php
@@ -71,7 +71,9 @@ trait ConvertAddresses {
protected function convertAddressList($envelope) {
$list = [];
foreach ($envelope as $t) {
- $list[] = $this->hordeToAssoc($t);
+ if ($t instanceof Horde_Mail_Rfc822_Address) {
+ $list[] = $this->hordeToAssoc($t);
+ }
}
return $list;
} That at least makes the list load again, not sure if the groups should be shown somehow or not. |
Awesome. Yeah we'll need a solution like that because there is no common base class for group addresses and regular ones. We should show groups too somehow. Does that class provide anything printable? |
There is But I guess that is caused by the sender, so you can use either one of them. |
I had the same problem, the fix from #53 (comment) works for me. Thanks @nickvergessen |
So just to note you as I reported this issue: #53 is the "solution". So after applying this patch it works even with the hidden recipients mails. There is no marker, that there are additional recipients though! |
Have same Problem and comment #53 help me. Just as info ... bevore upgrade from php 5.6 to php 7 it have working ... maybe php 5.6 don´t see this error and php 7 see it? WHatever, please fix it :-) Thx |
Same issue on NextCloud 11 and PHP 7.0, fixed with @nickvergessen solution, thank you! I hope this fix will be integrated soon. |
I had the same issue with Nextcloud 11.0.2.7 and PHP 7. Would be great, if the fix will be officially integrated. |
With the fix in place, all my emails do show again. |
Another duplicate, time to merge it @ChristophWurst ? |
Yeah time to open a PR @nickvergessen 😉 |
Oh, I didn't do that? I thought this is where we discuss.... |
What? 😁 |
@dennisheitmann this will be fixed with Mail 0.7.0. |
Tried to solve the issue but no success
Steps to reproduce
Expected behaviour
See all the messages and be able to answer and read them.
Actual behaviour
The app fails on certain folders to load the messages.
Mail app
Mail app version: (see apps admin page)
Mail 0.5.3
Mailserver or service: (e.g. Outlook, Yahoo, Gmail, Exchange,...)
dovecot (selfhosted) Thunderbird works well on those directories
Transport security - IMAP: (None, SSL, TLS, STARTTLS)
imap SSL/TLS
Transport security - SMTP: (None, SSL, TLS, STARTTLS)
not important
Number of accounts:
1
Mail app version build date: (only if you are using a Nightly Build)
Server configuration
Operating system:
Fedora 23
Web server:
Apache
Database:
mariadb
PHP version:
PHP 7.0.10
Version: (see admin page)
Nextcloud 10.0 RC1 (stable)
Updated from an older version or fresh install:
updated
List of activated apps:
The content of config/config.php:
Browser:
Firefox
Operating system:
Fedora 24
Logs
Web server error log
Server log (data/nextcloud.log)
The text was updated successfully, but these errors were encountered: