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 single role branch #2

Open
wants to merge 11 commits into
base: feature/role_support
from

Conversation

@jrosco
Copy link
Owner

jrosco commented Nov 9, 2019

No description provided.

jrosco added 11 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

liniting

add elif and nested elif statements

liniting
This reverts commit 26045ab.
This allow access to credeentails 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
Projects
None yet
1 participant
You can’t perform that action at this time.