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

non-configurable timeout for scp #397

Open
pulkomandy opened this issue Jan 10, 2022 · 3 comments
Open

non-configurable timeout for scp #397

pulkomandy opened this issue Jan 10, 2022 · 3 comments

Comments

@pulkomandy
Copy link

scp.py has a default but configurable timeout of 10 seconds: https://github.com/jbardin/scp.py/blob/master/scp.py#L103

In our case this is sometimes too short. Our ssh server takes a long time to reply to the connection request (probably because of misconfigured DNS causing DNS lookup timeouts). As a result we get a timeout error.

This timeout does not seem to be configurable without modifying sshlibrary. The connection timeout (used for Read Until) is not forwarded to scp.py, and there is no other way to set it (I don't know if it makes sense to use the same timeout). Only the transport is passed to the SCPClient constructor: https://github.com/robotframework/SSHLibrary/blob/master/src/SSHLibrary/pythonclient.py#L394

Does it make sense to reuse the timeout from the connection configuration for this? If that's appropriate, I think the fix would be reasonably easy to do.

@KirillYatsenko
Copy link

KirillYatsenko commented Dec 22, 2022

I have similar issue, have you found the workaround?

@KirillYatsenko
Copy link

KirillYatsenko commented Dec 23, 2022

Hi, I've created a simple MR for this issue: #423

@KirillYatsenko
Copy link

@pekkaklarck can we merge MR which resolves this issue?

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

No branches or pull requests

2 participants