Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Drop scp transport #53

Closed
agis opened this issue Apr 24, 2018 · 1 comment
Closed

Drop scp transport #53

agis opened this issue Apr 24, 2018 · 1 comment
Labels
idea wontfix This will not be worked on

Comments

@agis
Copy link
Contributor

agis commented Apr 24, 2018

It is hard to setup, not going to be used in production environments and overall rsync is a better solution. The only think we decided to add scp was for use in tests and as a more easy-to-setup default than rsync. However, it seems that it's not so easy since it still requires setting up SSH keys and also required more configuration to work as we want it (as opposed to rsync).

If scp doesn't provide the benefits we chose it for, we should drop it completely. For this, #52 should be done first so that we have an alternative (ie. running tests with rsync without having to setup an rsync daemon).

@agis
Copy link
Contributor Author

agis commented Jul 2, 2019

scp proved useful for testing and development scenarios, so I believe we should leave it as is. The cost of maintaining it is close to 0 anyway.

@agis agis closed this as completed Jul 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
idea wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant