Skip to content

Support for scp by installing openssh-client package#8

Closed
vladimirvivien wants to merge 1 commit intolinuxserver:masterfrom
vladimirvivien:master
Closed

Support for scp by installing openssh-client package#8
vladimirvivien wants to merge 1 commit intolinuxserver:masterfrom
vladimirvivien:master

Conversation

@vladimirvivien
Copy link
Copy Markdown

linuxserver.io


We welcome all PR’s though this doesn’t guarantee it will be accepted.

Description:

This change adds openssh-client tools to run server-side SCP commands.

Benefits of this PR and context:

Allow uses to run SCP commands

How Has This Been Tested?

Running SCP commands against a running container process.

Source / References:

@thelamer
Copy link
Copy Markdown
Member

I am not going to garuntee a merge as this might be outside of the intent of the container.

But you need to keep packages in alphabetical order and make the same modifications to the Dockerfile.aarch64 and the Dockerfile.armhf arm variants of this image.,

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

@vladimirvivien
Copy link
Copy Markdown
Author

@thelamer thanks for (possibly) considering this. I made changes as you requested.

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

@aptalca
Copy link
Copy Markdown
Member

aptalca commented Feb 3, 2020

Hi @vladimirvivien apologies for the wait.

We decided to add this as a docker mod, but had to figure out how to publish and host them first. There is now a mod for this at linuxserver/mods:openssh-server-openssh-client: https://github.com/linuxserver/docker-mods/tree/openssh-server-openssh-client

In your openssh-server container, set an environment variable DOCKER_MODS=linuxserver/mods:openssh-server-openssh-client and it will be installed during container start

@aptalca aptalca closed this Feb 3, 2020
@vladimirvivien
Copy link
Copy Markdown
Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants