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

Fix/checking roles #1

wants to merge 13 commits into
base: feature/multi_group_per_user


Copy link

jrosco commented Nov 9, 2019

No description provided.

jrosco added 13 commits Oct 25, 2019
Gem ffi set version to install, tries to install latest which requires
Ruby >2
* Added new "get_logged_in_user_role()" and "current_role()" functions

* Added new "require_role()" function does all the heavy work checking
if user is allowed to veiw/edit creds

* Updated endpoint /v1/user/email to be /v1/user/info which contains the
user email and role, this is used in the index.html file to show details
in top right corner

* Added settings for role support (enable/disable, property names etc)
cleanup lint errors
use kwargs with set_current_user() function (cleaner)
use double quotes
reduce line char count too long lint error
use lowercase for the group names in credential metadata

removed **kwargs and updated current_role function

Removed the **kwargs in set_current_user function
Update current_role function to return role "not set" message


add elif and nested elif statements

lower admin role string

This allow access to credentials that don't have a role set.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
1 participant
You can’t perform that action at this time.