Skip to content

Commit

Permalink
vdk-control-cli: Adopt new vdk-control-api-auth library (#840)
Browse files Browse the repository at this point in the history
With the release of the vdk-control-api-auth library, we now have a
stand-alone authentication package that is to be used by all Versatile
Data Kit components that need to authenticate against a third-party
service.

This change adopts the new authentication package in vdk-control-cli,
and is now used when issuing `vdk login` commands.

Additionally, authentication logic in vdk-control-cli which is overlapping
with such logic in vdk-control-api-auth is removed.

Testing Done: Installed the new changes locally and ran `vdk login` to verify
that the browser redirecs work as expected. Also, ran a docker container with
the latest changes to verify that authentication is possible even when there
is no access to a browser.

Signed-off-by: Andon Andonov <[email protected]>
  • Loading branch information
doks5 authored May 20, 2022
1 parent 139707b commit b2f2c9f
Show file tree
Hide file tree
Showing 8 changed files with 103 additions and 409 deletions.
1 change: 1 addition & 0 deletions projects/vdk-control-cli/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,6 @@ tabulate
# and use requests_oauthlib only, but for now it seems like there are some custom logic with the first
# OAuth2 provider which needs to be handled that way
urllib3>=1.26.5
vdk-control-api-auth

vdk-control-service-api==1.0.6
1 change: 1 addition & 0 deletions projects/vdk-control-cli/setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ install_requires =
tabulate
requests_oauthlib>=1.0
urllib3>=1.26.5
vdk-control-api-auth

# Require a specific Python version
python_requires = >=3.7, <3.11
Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

Loading

0 comments on commit b2f2c9f

Please sign in to comment.