Skip to content

feat: support keyboard-interactive - #188

Merged
caarlos0 merged 4 commits into
mainfrom
kbinter
May 30, 2023
Merged

caarlos0 merged 4 commits into
mainfrom
kbinter

Conversation

@caarlos0

@caarlos0 caarlos0 commented May 30, 2023 •

Copy link
Copy Markdown
Contributor

This PR adds keyboard-interactive auth method to both local and remote clients.

Limitations:

  • remote client will never echo anything because stdin is not in fact the stdin, but the parent's session stdin, which we don't have a fd for. It looks like this

refs #186

caarlos0 added 3 commits May 26, 2023 17:25
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the enhancement New feature or request label May 30, 2023
@caarlos0 caarlos0 self-assigned this May 30, 2023
@caarlos0
caarlos0 requested a review from aymanbagabas May 30, 2023 13:16

@aymanbagabas aymanbagabas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@caarlos0
caarlos0 merged commit 8da161a into main May 30, 2023
@caarlos0
caarlos0 deleted the kbinter branch May 30, 2023 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

⌂ 换网址