Skip to content
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

Mail not found, but whole source available #10361

Open
mat-m opened this issue Nov 16, 2024 · 2 comments · May be fixed by #10371
Open

Mail not found, but whole source available #10361

mat-m opened this issue Nov 16, 2024 · 2 comments · May be fixed by #10371

Comments

@mat-m
Copy link

mat-m commented Nov 16, 2024

Steps to reproduce

  1. I click on an email in the list (from 2 days ago, unread), and I get a "Not Found" error instead of the body, but headers are shown
  2. When I click the View Source in the mail headers, I get the full HTML body of the email

Expected behavior

Display the email body

Actual behavior

Error message in the page.

Also, in the logs, I have this error:

Exception
Call to a member function getEmail() on null in file '/var/www/html/custom_apps/mail/lib/Service/PhishingDetection/PhishingDetectionService.php' line 34

Mail app version

4.0.2

Nextcloud version

30.0.0

Mailserver or service

https://www.laposte.net/accueil

Operating system

Windows

PHP engine version

None

Nextcloud memory caching

No response

Web server

None

Database

None

Additional info

NC deployed through docker

@ChristophWurst
Copy link
Member

Dear @mat-m,
thank you very much for your report.
It seems there are some missing null checks in the phishing detection feature. Here's a patch: #10371.
I was not able to verify the changes myself, as the code paths only run for certain messages. I'd be happy if you can help test the changes.
See https://docs.nextcloud.com/server/30/admin_manual/issues/applying_patch.html#patching-apps for assistance.

@mat-m
Copy link
Author

mat-m commented Nov 30, 2024

Hello,

Patch applied, looks good. I skimmed through my mails and didn't find any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🏗️ In progress
Development

Successfully merging a pull request may close this issue.

2 participants