wbeuil/dctlenv

Version manager for driftctl

Hi there 👋

dctlenv's Issues

Change everything Cloudskiff related

Since the move from cloudskiff org to snyk org, some features are broken

Automatic installation in use cmd

It would be great if dctlenv use x.x.x can automatically install requested version instead of returning an error

GPG and checksum validation seems broken

And btw the checksum verification should be done by default, and not only when GPG signature is turned on.

Change verification of GPG signature

Need to change the verification of the signature with gpg --verify driftctl_SHA256SUMS.gpg driftctl_SHA256SUMS

Add dctlenv to the all-contributors specification

Add dctlenv to the all-contributors specification.

dctlenv use does not auto install if not version installed

To reproduce:

# Clone a new version of dctlenv
$ git clone ...
$ dctlenv use 0.5.0
# We got an error 'No versions of driftctl installed. Please install one with: dctlenv install [<version>]'

Bug on list command if the version display in the file is not installed

List all local versions even if the version file has a non-installed version displayed.

We should be able to uninstall a version even if the install failed at the end

$ dctlenv list
* 0.11.0 (set by /Users/wbeuil/.dctlenv/version)
  0.10.0
  0.9.1
  0.9.0
  0.5.0

$ dctlenv uninstall 0.10.0
Uninstall driftctl v0.10.0
driftctl v0.10.0 is successfully uninstalled

$ dctlenv uninstall 0.9.1

$ dctlenv list
* 0.11.0 (set by /Users/wbeuil/.dctlenv/version)
  0.9.1
  0.9.0
  0.5.0

Drop `DCTLENV_PGP`

I think we could drop the need of DCTLENV_PGP.
We can use an alternative logic that does things automatically.

Have something like that in mind:

• check if version is above v0.9.1
• check if gpg binary is present
• check if key 2776 6600 5A7F 01D4 84F6 376D ACC7 76A7 9C82 4EBD is present (based on the fingerprint please not mail or ID it can be insecure)
• If all theses conditions are met, then do a gpg check and display a green message
• Else display a warning message in yellow on stderr to say that we are unable to verify the authenticity of the binary

Add tests

Add unit/acceptance tests

Allow to install the latest version

It could be great to have a dctlenv install-latest or dctlenv upgrade or dctlenv use latest to automatically download, install and use latest available version 😉

Add doc for environment variables

Write in the README about env variables.

Add CI/CD pipeline with GitHub Actions

Run make which will run our test suite on each push and pull_request.

Make version param more flexible

It could be cool that both v0.4.0 and 0.4.0 can work as version input with dctlenv.

With something like this you can normalize every version number :

• echo "v0.4.0" | sed -s 's/^v//' => 0.4.0
• echo "0.4.0" | sed -s 's/^v//' => 0.4.0

Actually we got a message which is pretty confusing as version number are inconsistent:

Should verify checksum on install

snyk/driftctl#190 will add checksum alongside binaries.

We should verify it on install !