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

Remove Client.set_missing_host_key_policy class-vs-instance footgun #857

Closed
bitprophet opened this issue Dec 2, 2016 · 0 comments
Closed

Remove Client.set_missing_host_key_policy class-vs-instance footgun #857

bitprophet opened this issue Dec 2, 2016 · 0 comments

Comments

@bitprophet
Copy link
Member

@bitprophet bitprophet commented Dec 2, 2016

I don't see any reason why the mentioned method shouldn't accept, and then auto instantiate, policy classes as well as instances. This API has always felt bizarre to me and it very frequently trips users up.

Instead of adding an FAQ we should just make it do the smart thing.

@bitprophet bitprophet added this to the 2.1 milestone Dec 2, 2016
bitprophet added a commit that referenced this issue Jun 6, 2017
bitprophet added a commit that referenced this issue Jun 6, 2017
@bitprophet bitprophet closed this in a2e54d5 Jun 6, 2017
bitprophet added a commit that referenced this issue Jun 6, 2017
dkhapun pushed a commit to cyberx-labs/paramiko that referenced this issue Jun 7, 2018
dkhapun pushed a commit to cyberx-labs/paramiko that referenced this issue Jun 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant