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.

Sign up for GitHub

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

Jump to bottom

Adds emittion of user authentication success event. #76

Merged
merged 2 commits into from
Mar 3, 2021
Merged

Adds emittion of user authentication success event. #76

merged 2 commits into from
Mar 3, 2021

Commits on Mar 3, 2021

  1. Adds emittion of user authentication success event. 

    Motivation:
Currently there is now way to wait for successful SSH session
establishment, the only way to continue is to try to create a child
channel. Given that most clients are expecting the session to be active
when both key exchange and user authenticaiton complete, I propose to
emit UserAuthSuccessEvent.  That way client can wait for that event or
fail if there is an error in the pipeline.

Modifications:
 - Adds UserAuthSuccessEvent event
 - Adds .event case to StateMachineInboundProcessResult
 - Adds emittion of event if StateMachineInboundProcessResult is .event
 - Adds two tests

Result:
Closes apple#39
    @artemredkin
    artemredkin committed Mar 3, 2021
    Configuration menu
    Copy the full SHA
    62077dc View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into add_userauth_success_event

    @artemredkin
    artemredkin authored Mar 3, 2021
    Configuration menu
    Copy the full SHA
    97fcf8d View commit details
    Browse the repository at this point in the history