Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Bugfixes to the SCP command in Windows + Add trusted CA command #4689

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

Conversation

deblasis
Copy link

Description

Hi there,

I understand the status of the project, however, this PR addresses issues with the SCP command in Windows (#4444, #4498)

Also, it adds a new feature that allows to upload and trust CA Certificates as described in #1799
Test coverage is almost 100% on that.

Best regards

Related issue(s)

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "add_trusted_ca_command" git@github.com:deblasis/machine.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842358449896
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Signed-off-by: Alessandro De Blasis <alex@deblasis.net>
This is to avoid redundancy at the expense of readability
Optional commit depending on the team's view on the matter

Signed-off-by: Alessandro De Blasis <alex@deblasis.net>
Signed-off-by: Alessandro De Blasis <alex@deblasis.net>
the full path could have spaces on windows, better safe than sorry

Signed-off-by: Alessandro De Blasis <alex@deblasis.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
2 participants