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

version 1 of more granular permission level. #158

Open
wants to merge 73 commits into
base: master
Choose a base branch
from

Conversation

grugna
Copy link

@grugna grugna commented May 27, 2020

Improvements

This pull request allows peregrine to filter the results of a query one level under Project. This will allow to assign more granular permission to a user.

/program/{program}/project/{project}/{node}/{submitter_id}

The node name and model is configurable in the environment variable since it can differ among different dictionaries.

grugna added 10 commits May 27, 2020 10:17
new version of cryptography breaks the build, rolling back to previous version: cryptography==2.8
Increased granularity for permission to read resources from project level to subject level
Fix for the case where the configuration containing the name of the subject node is missing.
function name changed needed to update it
@lgtm-com
Copy link

lgtm-com bot commented May 28, 2020

This pull request introduces 1 alert when merging 5d17fd8 into 1d9341a - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Jun 19, 2020

This pull request introduces 1 alert when merging 01f4570 into 1d9341a - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Jun 24, 2020

This pull request introduces 1 alert when merging 07b8afa into 1d9341a - view on LGTM.com

new alerts:

  • 1 for Unused local variable

merge uc-cdis peregrine master
@lgtm-com
Copy link

lgtm-com bot commented Aug 17, 2020

This pull request introduces 1 alert when merging 1e8cfe9 into 4da2cfb - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Aug 17, 2020

This pull request introduces 1 alert when merging 29472b6 into 4da2cfb - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Sep 1, 2020

This pull request introduces 1 alert when merging a9f3f22 into 4da2cfb - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Oct 2, 2020

This pull request introduces 1 alert when merging e6a3ae6 into 4da2cfb - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Oct 23, 2020

This pull request introduces 1 alert when merging 1210673 into 4da2cfb - view on LGTM.com

new alerts:

  • 1 for Unused local variable

A user without any acces was able to see al the data since the filter was applied only if the user had partial access
@lgtm-com
Copy link

lgtm-com bot commented Oct 28, 2020

This pull request introduces 1 alert when merging 9d0a4a2 into 4da2cfb - view on LGTM.com

new alerts:

  • 1 for Unused local variable

…out_patch

Extend arborist requests timeout patch
chore(logging): add query to timing log (uc-cdis#216)
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