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

Call stat before opening file for reading #562

merged 2 commits into from Nov 4, 2015


Copy link

@torkil torkil commented Jul 23, 2015

Calling stat after opening a file for reading will in some server
configurations(?) lead to IOError: The message 'filename' is
not extractable.

Calling stat from inside the prefetch-body has led users to
receive IOError: The message [<filename>] is not extractable.
@torkil torkil changed the title prefetch now requires file_size to be passed in as a parameter Call stat before opening file for reading Jul 23, 2015
@torkil torkil closed this Jul 23, 2015
@torkil torkil reopened this Jul 23, 2015
Copy link

@bitprophet bitprophet commented Jul 30, 2015

Do you have more info on what "some server configurations" means exactly - i.e., is there a reproducible way we can test this out?

The change seems reasonable at a glance either way, though, so thanks!

Copy link
Contributor Author

@torkil torkil commented Jul 30, 2015

The erroneous server configuration is something which have been pointed out in issues I think are related see, #194 #530

I also think this is due, at least in some way, to

@bitprophet bitprophet added this to the 1.13.4 / 1.14.3 / 1.15.3 milestone Aug 19, 2015
@bitprophet bitprophet modified the milestones: 1.15.3, 1.16 Oct 2, 2015
@bitprophet bitprophet modified the milestone: 1.16 Nov 3, 2015
@bitprophet bitprophet merged commit 49072f3 into paramiko:master Nov 4, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
dkhapun pushed a commit to cyberx-labs/paramiko that referenced this pull request Jun 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants