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

Implement access management #196

Open
uniqueg opened this issue Jul 21, 2020 · 0 comments
Open

Implement access management #196

uniqueg opened this issue Jul 21, 2020 · 0 comments
Labels
priority: medium Medium priority type: auth Related to authorization or authentication type: security Related to security workload: days Likely takes days to resolve

Comments

@uniqueg
Copy link
Member

uniqueg commented Jul 21, 2020

Is your feature request related to a problem? Please describe.

Currently, access to cwl-WES is granted to everyone with a valid bearer token, provided that the authorization_required option is enabled in the app configuration (cwl_wes/config/app_config.yaml). However, resources are only restricted inasmuch that users can only perform GET operations on workflow runs they have created themselves via POST /runs. To protect cwl-WES resources from unprivileged users, access to both POST and GET methods on /runs and derived endpoints should be granted only if a given user has the necessary permissions.

Describe the solution you'd like

Ideally, access management should follow the GA4GH Passport specification to ensure that only privileged users can access resources.

Users are granted access to a resource only if their token contains the necessary visa.

Describe alternatives you've considered

Alternatively, or in addition, an access control management endpoint should be designed that allows devops/system admins to set permissions individually for each user and/or user groups (e.g. ELIXIR AAI groups).

Additional context

N/A

@uniqueg uniqueg added priority: medium Medium priority type: auth Related to authorization or authentication type: security Related to security workload: days Likely takes days to resolve labels Jul 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Medium priority type: auth Related to authorization or authentication type: security Related to security workload: days Likely takes days to resolve
Projects
None yet
Development

No branches or pull requests

1 participant