-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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 when sharing to user with email #12597
Comments
GitMate.io thinks possibly related issues are #4334 (Improve sharing email information), #5658 (share by email - password email not sent - solved), #5047 (Can't password protect shares because password email is sent from account email address), #10132 (Allow restoring of deleted user shares), and #8944 (Automatic email sending for shared folder sends users main password!). |
I had a quick look at the issue. The problem is that we try to send a mail to the recipient, informing them that a file was shared: https://github.com/nextcloud/server/blob/master/lib/private/Share20/Manager.php#L679 This could be fixed quite easy with some error handling or a check if mail is configured on the server... But I don't understand why we send this mail at all in the sharing code? Istn't this something activities should handle? In the activities settings you can configure mail notifications for shares, in this case you would get them twice. Further as far as I understand there is no settings where a user/admin could configure if this mails should be send or not, by default it is always set to true. At the moment I tend to just remove it completely... Do I miss something? |
I will have a look at it and propose a PR |
This also happens on a fresh NC 14 installation |
userX
userX
userX
Now: You get a nice error message.
You should not get an error message.
The text was updated successfully, but these errors were encountered: