Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scp: add option to automatically accept signature #204

Closed
wants to merge 1 commit into from
Closed

scp: add option to automatically accept signature #204

wants to merge 1 commit into from

Conversation

teknoraver
Copy link

Add an -y option which automatically accepts the host key without checking as we did with ssh in commit cd0a088

Add an '-y' option which automatically accepts the host key without checking
as we did with ssh in commit cd0a088
@Ansuel
Copy link

Ansuel commented Feb 7, 2023

@mkj sorry for the tag. Any problem with this simple patch?

@mrkiko
Copy link

mrkiko commented Feb 21, 2023

Yeah - would be very nice to see this merged :)

@mkj
Copy link
Owner

mkj commented Feb 22, 2023

I wonder if it would be better to have -o StrictHostKeyChecking=off rather than adding another extra flag?

@teknoraver
Copy link
Author

That's to be coherent with ssh, which already supports the -y flag

@okashany
Copy link

okashany commented Jun 1, 2023

Hi, is there any reason why this patch was not merged?

@mkj
Copy link
Owner

mkj commented Jun 28, 2023

I'm reluctant to let scp diverge from the flags in upstream OpenSSH - I don't really want to maintain it myself (even OpenSSH have mostly given up on it).

Generally I think the argument lists of dropbear and dbclient are getting unwieldy, so options that aren't usually typed by hand should go under -o instead. Of course dropbear itself doesn't have -o support yet, but one day it will.

@stokito
Copy link
Contributor

stokito commented Dec 24, 2023

Related to #41 that adds -o and #269 that adds to the dbclient -o StrictHostKeyChecking

@mkj
Copy link
Owner

mkj commented Apr 3, 2024

scp -o stricthostkeychecking should work since #269

@mkj mkj closed this Apr 3, 2024
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.

6 participants