Giter Club home page Giter Club logo

Comments (17)

nickvergessen avatar nickvergessen commented on May 13, 2024 3

Got this too, happens when you receive an email with a hidden recipient list.
For me there is a Group "Verborgene_Empfaenger" on that mail.

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.

from mail.

enaut avatar enaut commented on May 13, 2024

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!
If I click 'edit as new' and send the mail to myself the problem persists!

What should I do next? I could send someone the problematic email but I don't want it posted here!

Any help is appreciated

from mail.

ChristophWurst avatar ChristophWurst commented on May 13, 2024

Horde_Mail_Rfc822_Group

@irgendwie πŸ˜‰

from mail.

ChristophWurst avatar ChristophWurst commented on May 13, 2024

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?

from mail.

nickvergessen avatar nickvergessen commented on May 13, 2024

There is $t->groupname, $t->groupname_encoded and $t->label but all 3 are Verborgene_Empfaenger for me, so not really a nice printed version....

But I guess that is caused by the sender, so you can use either one of them.

from mail.

matzex avatar matzex commented on May 13, 2024

I had the same problem, the fix from #53 (comment) works for me. Thanks @nickvergessen

from mail.

enaut avatar enaut commented on May 13, 2024

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!

from mail.

chrissooo avatar chrissooo commented on May 13, 2024

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

from mail.

roipoussiere avatar roipoussiere commented on May 13, 2024

Same issue on NextCloud 11 and PHP 7.0, fixed with @nickvergessen solution, thank you!

I hope this fix will be integrated soon.

from mail.

Tarry91 avatar Tarry91 commented on May 13, 2024

I had the same issue with Nextcloud 11.0.2.7 and PHP 7. Would be great, if the fix will be officially integrated.

from mail.

pieter-groeneweg avatar pieter-groeneweg commented on May 13, 2024

With the fix in place, all my emails do show again.

from mail.

nickvergessen avatar nickvergessen commented on May 13, 2024

Another duplicate, time to merge it @ChristophWurst ?

from mail.

ChristophWurst avatar ChristophWurst commented on May 13, 2024

Yeah time to open a PR @nickvergessen πŸ˜‰

from mail.

nickvergessen avatar nickvergessen commented on May 13, 2024

Oh, I didn't do that? I thought this is where we discuss....

from mail.

ChristophWurst avatar ChristophWurst commented on May 13, 2024

What? 😁

from mail.

dennisheitmann avatar dennisheitmann commented on May 13, 2024

Same issue here on Nextcloud 12.0.0 and Mail App 0.6.4 on Apache2 / PHP7.

grafik

My nextcloud is called owncloud (for historical reasons)...

from mail.

ChristophWurst avatar ChristophWurst commented on May 13, 2024

@dennisheitmann this will be fixed with Mail 0.7.0.

from mail.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.