search

zorn-v / nextcloud-social-login

GNU Affero General Public License v3.0
198 stars 137 forks source link

Group mapping: map towards several groups #340

Closed arkanovicz closed 1 year ago

arkanovicz commented 2 years ago

Looking at the source, it would not be too difficult to allow a mapping from one source group to several nextcloud groups.

One could specify it by listing coma separated values in the destination mapping field. This new feature can easily be backward compatible.

zorn-v commented 2 years ago

Why do not put user in several groups on provider side ? I can not imagine scenario when proposed feature will be useful. But it may be more inconvenient than now - currently "local" group is select box of existing NC groups.

Blackclaws commented 2 years ago

I would definitely second what zorn-v said. You can usually set up mappers that map a single group to multiple groups before putting them into the OIDC token. That way you don't create an arbitrary mapping in nextcloud and keep your source of truth closer to the user store.