-
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
This app does not respect the "Allow sharing with groups"-setting #6017
Comments
Hey @pierreozoux 👋 I need some help with your bug report. Sharing Settings Users & Groups Test Case 1
* (Nextcloud) = Group in Nextcloud Test Case 2
Night's Watch (Nextcloud) = Group in Nextcloud Test Case 3
Robb Stark (robb@stark.got) is a direct match from contacts then Test Case 1, 2 and 3 look okay. I also had a quick look at the auto completion service. We use three sources: Contacts, Groups and collected addresses. Can you share some more details how to reproduce it 🤔 |
Hi @kesselb This looks definitely like a duplicate of #6148 I'll answer for @pierreozoux Its weird I cannot replicate on a fresh install but on our current install we do have the issue of the autocompletion settings being ignored... If I allow auto-completion for users in the same groups: Can it be something that is messed up during an upgrade when there was this issues ? We still have some stuff in the db that should be cleaned up ? Nextcloud version 23.0.7 Thanks |
Hi @kesselb, As I understand it, the ContactsManager (lib/private) is not responsible for filtering the results. Then, I don't see how the code would filter by groups as it only filters on mail/lib/Service/ContactsIntegration.php Lines 56 to 63 in ca3f929
I believe we need something like this. Have I missed something ? Thanks ! |
Label dropped because it was a duplicate of the attribute at https://github.com/orgs/nextcloud/projects/61. The priority does not change. |
Expected behavior
Given the following settings
When you write a new email, you shouldn't be proposed autocompletion emails from users from groups you are not in.
Actual behavior
It does propose you all the emails of the instance.
Steps to reproduce
Mail app version
1.11.6
Mailserver or service
Not relevant.
Operating system
Not relevant.
PHP engine version
Other
Web server
Other
Database
Other
Additional info
Relates to this bug:
nextcloud/server#25390
On shared instance, this is a privacy issue.
The text was updated successfully, but these errors were encountered: