common-workflow-language / common-workflow-language/workflow-service

Add HTTP basic auth

Open
#46 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
39
Forks
21
PR merge metrics
No merged PRs in 30d

Description

So that a username/password combination can be provided by environment variables to the service. All endpoints (except service info?) should be under auth.

Close with tests that demonstrate the server running under basic auth using a client that can use the username/pass combination.

For examples of making pluggable auth with connexion see dos_connect:

https://github.com/ohsu-comp-bio/dos_connect/blob/master/dos_connect/server/basic_authorizer.py

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the service's endpoint and server setup, then compare the pluggable Connexion auth example in dos_connect/server/basic_authorizer.py. Add tests that run the server with credentials supplied through environment variables and authenticate a client against protected endpoints, while confirming the service-info exception if retained.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, authentication, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.