Skip to content

Fix keyboard-interactive password fallback#143

Closed
shcheklein wants to merge 3 commits intotreeverse:mainfrom
shcheklein:fix-kbdint-password-fallback
Closed

Fix keyboard-interactive password fallback#143
shcheklein wants to merge 3 commits intotreeverse:mainfrom
shcheklein:fix-kbdint-password-fallback

Conversation

@shcheklein
Copy link
Copy Markdown

@shcheklein shcheklein commented Apr 22, 2026

Fixes issue of DVC not picking up SSH password from DVC config.

  • return NotImplemented from kbdint_auth_requested() when a password is already configured
  • let asyncssh reuse the configured password for keyboard-interactive password challenges
  • add a focused unit test for the callback contract

Note: AI was used, I've reviewed and experimented a bit with Docker image.

@shcheklein
Copy link
Copy Markdown
Author

Closing this, I think #141 is probably better.

@shcheklein shcheklein closed this Apr 23, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

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.

1 participant