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

Feature Request: Confirm constraint #15

Open
mhofman opened this issue Feb 27, 2019 · 1 comment
Open

Feature Request: Confirm constraint #15

mhofman opened this issue Feb 27, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@mhofman
Copy link

mhofman commented Feb 27, 2019

It would be great to support the confirm mechanism that exists in the SSH agent protocol.

Currently when loading a key through ssh-add with the -c option, the key will actually be used silently without any prompt.

I looked through the golang crypto source code, and it looks like the agent server parses the constraint and sets a ConfirmBeforeUse flag on the key, but the keyring.add method simply ignores it.

From a UI persepective, I'd want the options page to set the constraint when adding a key and / or when loading it.
For the prompt, the extension could show a notification (with require interaction set), to ask to either confirm or deny.

@ralimi
Copy link
Collaborator

ralimi commented Aug 9, 2022

I can understand the use case here, and the desire for this to be a fully-functioning SSH agent. However, Go's current implementation doesn't really appear to be more suitable for the more generic use cases, so doing this would be a much larger undertaking.

As another example, #25 is a problem facing keys added directly from a connected ssh client.

Keeping this open in case something comes to mind here.

@ralimi ralimi added the enhancement New feature or request label Aug 28, 2022
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

No branches or pull requests

2 participants