Skip to content

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Mar 25, 2020

In case auto accepting is enabled (the default). Users that are newly
added to a group should not have to accept those shares.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

In case auto accepting is enabled (the default). Users that are newly
added to a group should not have to accept those shares.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added bug 3. to review Waiting for reviews labels Mar 25, 2020
@rullzer rullzer added this to the Nextcloud 19 milestone Mar 25, 2020
@rullzer
Copy link
Member Author

rullzer commented Mar 25, 2020

/backport to stable18

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't testy but makes sense

@kesselb
Copy link
Contributor

kesselb commented Mar 25, 2020

May fix #19520

@rullzer
Copy link
Member Author

rullzer commented Mar 25, 2020

May fix #19520

probably

@rullzer
Copy link
Member Author

rullzer commented Mar 25, 2020

Testing is easy.

  1. Create a new group
  2. Share a file to this group
  3. Create a new user
  4. Add this user to the group
  5. Login as this user

before: Having to accept the group share

after: not having to accept the group share

@rullzer rullzer merged commit 61cf7be into master Mar 25, 2020
@rullzer rullzer deleted the fix/auto_accept_group_shares_new_users branch March 25, 2020 12:05
@backportbot-nextcloud
Copy link

backport to stable18 in #20161

@skjnldsv
Copy link
Member

10000th PR to be merged! 🎉

@kklem0
Copy link
Contributor

kklem0 commented Apr 28, 2020

Quick question @rullzer , does this work with LDAP group as well?

@jsalatiel
Copy link

I am running 18.06 with ldap/saml activated. New users added to the group can not see the shared folders to the group even though they have settings->sharing-> "Accept user and groups share by default" selected.
They must go to Shares -> pending Shares and accept to make the folder appear.

@tofuSCHNITZEL
Copy link
Contributor

yes same with me and 18.0.6

@liejuntao001
Copy link

I am running 18.06 with ldap/saml activated. New users added to the group can not see the shared folders to the group even though they have settings->sharing-> "Accept user and groups share by default" selected.
They must go to Shares -> pending Shares and accept to make the folder appear.

I have the same setup 18.06, but the new user has nothing in Shares -> pending Shares.
Before upgrading to 18.06, when it was 18.04, the new user could see the Shares -> pending Shares to accept the sharing.
I think this is a breakage.

@demlak
Copy link

demlak commented Nov 14, 2020

Still not working on new ldap-users in nextcloud 19.0.4
Already shared files and folders are still pending shares.

@smirta
Copy link

smirta commented Nov 16, 2020

Does your cron.php-Job run? If not, you'll need to make sure that it is running with the latest php-runtime. It fails to run with php7.2. You'll need to change to php7.3. We had to adjust the systemd-unit. E.g.:

ExecStart=/opt/rh/rh-php72/root/usr/bin/php -f /var/www/html/nextcloud/cron.php

to

ExecStart=/opt/rh/rh-php73/root/usr/bin/php -f /var/www/html/nextcloud/cron.php

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.