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

default host key policy to RejectPolicy #2280

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Sep 6, 2023

  1. default host key policy to RejectPolicy

    It's a really bad idea to silently accept new keys. It's not how SSH
    works, and it's not how Fabric *should* work. If we'd work like SSH,
    we'd *prompt* the user, but the WarningPolicy doesn't do that, it just
    warns, and doesn't prompt.
    
    At the very least those policies should noisily warn the user when the
    host keys get changed, but that's something Paramiko must do.
    
    An alternative to this would be WarningPolicy but I think it's too
    light and possibly too late: when you warn, Fabric probably has time
    to run the bad stuff before the operator notices and interrupts.
    anarcat committed Sep 6, 2023
    Configuration menu
    Copy the full SHA
    083552c View commit details
    Browse the repository at this point in the history