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

Rework dial and accept to return Session #565

Closed
wants to merge 12 commits into from
Closed

Rework dial and accept to return Session #565

wants to merge 12 commits into from

Commits on Sep 20, 2021

  1. Adds quick test run that "only" takes about 20 seconds

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    bb08eda View commit details
    Browse the repository at this point in the history

  2. Rename Transport.accept() -> Transport.acceptStream()

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    7c3f4a6 View commit details
    Browse the repository at this point in the history

  3. Adds Transport.accept(), which returns a Session

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    870cb18 View commit details
    Browse the repository at this point in the history

  4. Adds TcpTransport.accept(), which returns TcpSession 

    Also adds list of Sessions to TcpTransport, which will
replace clients list eventually.
    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    901dea3 View commit details
    Browse the repository at this point in the history

  5. Adds WsTransport.accept(), which returns WsTransportSession 

    Also adds list of Sessions to WsTransport, which will
replace clients list eventually.
    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    cad3f95 View commit details
    Browse the repository at this point in the history

  6. Implement acceptStream using accept() and getStream()

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    58c053d View commit details
    Browse the repository at this point in the history

  7. Rename Transport.dial() -> Transport.dialStream()

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    f4902a3 View commit details
    Browse the repository at this point in the history

  8. Implement dialStream using dial() and getStream()

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    4241c5d View commit details
    Browse the repository at this point in the history

  9. Remove clients and connHandler from tcp and ws transport 

    These have been replaced by sessions and sessionHandler.
    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    a927bb2 View commit details
    Browse the repository at this point in the history

  10. Handle Windows ERROR_NETNAME_DELETED in accept() 

    This error code indicates that server socket is already
closed, so we raise a TransportClosedError.

Fixes CI failure in Windows.
    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    a87d97c View commit details
    Browse the repository at this point in the history

  11. Add direction parameter to Session.getStream() 

    Quic allows opening outgoing or accepting incoming streams.
    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    eea29da View commit details
    Browse the repository at this point in the history

  12. Keep implicit Sessions in memory and close them properly.

    @markspanbroek
    markspanbroek committed Sep 20, 2021
    Configuration menu
    Copy the full SHA
    0c5b832 View commit details
    Browse the repository at this point in the history