Please add 'Allow login only from specified domains' option to yandex config too

zorn-v / nextcloud-social-login

GNU Affero General Public License v3.0

198 stars 137 forks source link

Please add 'Allow login only from specified domains' option to yandex config too #324

Closed lierdakil closed 2 years ago

lierdakil commented 2 years ago

AFAIU, it should be enough to duplicate this block, changing the provider name to yandex

https://github.com/zorn-v/nextcloud-social-login/blob/f071ad6e19dcaa236744c5cae8a9dbdb1e56e1ab/src/components/Settings.vue#L90-L96

The back-end apparently already has everything required in place (manually setting "auth_params" for yandex in oauth_providers config works as expected, but it gets reset when settings are modified in the web ui)

zorn-v commented 2 years ago

Check later

lierdakil commented 2 years ago

Ping. What's the status on this? Would creating a pull request help?

zorn-v commented 2 years ago

Check v4.13.0

lierdakil commented 2 years ago

Yep, that works. Thanks!