Skip to content
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

I have made the plugin more customizable #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

erikthegamer1242
Copy link

@erikthegamer1242 erikthegamer1242 commented Jul 24, 2023

I at home run Authelia, which uses different scopes for some reason, so I have made it so they can be more customized, plus you can now have as many user and admin roles defined per SSO in the create menu.

1. Customizable roles scopes and all scopes:

  • Now in the config.ini file there are two more variables that can be set:
OAUTH_ROLES_SCOPE = ""
OAUTH_ALL_SCOPES = ""
  • The first one controls what scope will be used for getting user roles, specifically for Authelia this is groups, if not set the default will be roles
  • The second controls all of the scopes this plugin will try to access to get various data, for eg. you need the special email scope for Authelia to get the user email address. If not set the default will be profile roles

2. You can now assign what roles you want to be admin and user, it's not hardcoded anymore:

  • I have added two new fields to the form, the database and the user role if statement
  • The create SSO UI now shows those two new fields with the data being populated as admin and user respectably so the users just clicking 'next, next, finish' don't get confused

image

The only thing to keep in mind is that the user will be assigned either an admin or user role based on what their Identity Provider gives as the first role so if you have user, admin you will be assigned a user role

…O Client, also the input field are shown in the Create tab
… many SSO roles for admins and users, plus the scopes can now be set using config, because I had some troubles adjusting to Authelia
…enght is stil technically larger then one in my case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant