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

svg logo prevents emails from being sent #11453

Closed
errotu opened this issue Sep 29, 2018 · 3 comments
Closed

svg logo prevents emails from being sent #11453

errotu opened this issue Sep 29, 2018 · 3 comments
Labels

Comments

@errotu
Copy link

errotu commented Sep 29, 2018

Steps to reproduce

  1. Upload SVG file as logo in the Theming-area of the backend
  2. Test your email settings
  3. Emails are not being sent, instead the "Senden" (German)-notification turns red. Nothing more happens.

Expected behaviour

Emails should be sent even if PNG conversion fails (as the log indicates). In these cases, I would recommend that the emails don't contain any logo and a notification is shown to the admin.

Actual behaviour

Emails are not sent.

Server configuration

PHP version: 7.1

Nextcloud version: 14.0.1

Updated from an older Nextcloud/ownCloud or fresh install: Updated from NC 13.06

Nextcloud log (data/nextcloud.log)

Nextcloud log
{"reqId":"W69U2SUR4CYAAGeohfEAAAAA","level":3,"time":"2018-09-29T10:32:58+00:00","remoteAddr":"79.249.65.182","user":"CTO","app":"index","method":"POST","url":"\/index.php\/settings\/admin\/mailtest","message":{"Exception":"TypeError","Message":"Argument 2 passed to OC\\Log::info() must be of the type array, string given, called in \/home\/www\/cloud\/apps\/theming\/lib\/ImageManager.php on line 120","Code":0,"Trace":[{"file":"\/home\/www\/cloud\/apps\/theming\/lib\/ImageManager.php","line":120,"function":"info","class":"OC\\Log","type":"->","args":["The image was requested to be no SVG file, but converting it to PNG failed.","not authorized `\/tmp\/magick-26536nA5YFtSU1GyE' @ error\/constitute.c\/ReadImage\/412"]},{"file":"\/home\/www\/cloud\/apps\/theming\/lib\/ThemingDefaults.php","line":211,"function":"getImage","class":"OCA\\Theming\\ImageManager","type":"->","args":["logo",false]},{"file":"\/home\/www\/cloud\/lib\/public\/Defaults.php","line":187,"function":"getLogo","class":"OCA\\Theming\\ThemingDefaults","type":"->","args":[false]},{"file":"\/home\/www\/cloud\/lib\/private\/Mail\/EMailTemplate.php","line":384,"function":"getLogo","class":"OCP\\Defaults","type":"->","args":[false]},{"file":"\/home\/www\/cloud\/settings\/Controller\/MailSettingsController.php","line":150,"function":"addHeader","class":"OC\\Mail\\EMailTemplate","type":"->","args":[]},{"file":"\/home\/www\/cloud\/lib\/private\/AppFramework\/Http\/Dispatcher.php","line":166,"function":"sendTestMail","class":"OC\\Settings\\Controller\\MailSettingsController","type":"->","args":[]},{"file":"\/home\/www\/cloud\/lib\/private\/AppFramework\/Http\/Dispatcher.php","line":99,"function":"executeController","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[{"__class__":"OC\\Settings\\Controller\\MailSettingsController"},"sendTestMail"]},{"file":"\/home\/www\/cloud\/lib\/private\/AppFramework\/App.php","line":118,"function":"dispatch","class":"OC\\AppFramework\\Http\\Dispatcher","type":"->","args":[{"__class__":"OC\\Settings\\Controller\\MailSettingsController"},"sendTestMail"]},{"file":"\/home\/www\/cloud\/lib\/private\/AppFramework\/Routing\/RouteActionHandler.php","line":47,"function":"main","class":"OC\\AppFramework\\App","type":"::","args":["OC\\Settings\\Controller\\MailSettingsController","sendTestMail",{"__class__":"OC\\AppFramework\\DependencyInjection\\DIContainer"},{"_route":"settings.MailSettings.sendTestMail"}]},{"function":"__invoke","class":"OC\\AppFramework\\Routing\\RouteActionHandler","type":"->","args":[{"_route":"settings.MailSettings.sendTestMail"}]},{"file":"\/home\/www\/cloud\/lib\/private\/Route\/Router.php","line":297,"function":"call_user_func","args":[{"__class__":"OC\\AppFramework\\Routing\\RouteActionHandler"},{"_route":"settings.MailSettings.sendTestMail"}]},{"file":"\/home\/www\/cloud\/lib\/base.php","line":987,"function":"match","class":"OC\\Route\\Router","type":"->","args":["\/settings\/admin\/mailtest"]},{"file":"\/home\/www\/cloud\/index.php","line":42,"function":"handleRequest","class":"OC","type":"::","args":[]}],"File":"\/home\/www\/cloud\/lib\/private\/Log.php","Line":179,"CustomMessage":"--"},"userAgent":"Mozilla\/5.0 (X11; Ubuntu; Linux x86_64; rv:62.0) Gecko\/20100101 Firefox\/62.0","version":"14.0.1.1"}
@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #5658 (share by email - password email not sent - solved), #5047 (Can't password protect shares because password email is sent from account email address), #1916 (discussion: remove "send email" option for public links), #11232 (Verification email not sending NextCloud Forums), and #7310 (Sending pgp encrypted Emails).

@kesselb
Copy link
Contributor

kesselb commented Sep 29, 2018

Could this be a duplicate of #11206? If so could you close this issue than?

@errotu
Copy link
Author

errotu commented Sep 29, 2018

It is indeed. Sorry for the duplicate!

@errotu errotu closed this as completed Sep 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants