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

NIOSSHClientUserAuthenticationDelegate Improvement Request: Result Reporting #65

Open
aaronvegh opened this issue Jan 21, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@aaronvegh
Copy link

The NIOSSHClientUserAuthenticationDelegate provides a single method, nextAuthenticationRequest, for providing offers for authenticating with a remote host. In the golden path, an auth request is successful and everyone is happy. However, in situations where the request fails, either because of incorrect credentials or an unexpected failure on the remote end, it would be useful for the delegate to receive a notification to that effect. As things stand now, the only way to know an offer fails is if the method is called again. That type of inference is something I'd rather not rely upon.

In my research of this repository, I came across NIOSSHUserAuthenticationOutcome; I wonder if using that here would be helpful?

@Lukasa Lukasa added the enhancement New feature or request label Jan 21, 2021
@Lukasa
Copy link
Collaborator

Lukasa commented Jan 21, 2021

Yeah, we should add more feedback to the client user auth delegate, especially as we so tightly control its execution environment.

@Joannis
Copy link
Collaborator

Joannis commented Oct 31, 2023

@Lukasa do you have a specific design in mind for this API? I'm happy to make a PR for it.

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

3 participants