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

More ergonomic support for using a unix socket with a tonic server #1

Merged
merged 3 commits into from
Dec 8, 2021
Merged

More ergonomic support for using a unix socket with a tonic server #1

merged 3 commits into from
Dec 8, 2021

Commits on Dec 7, 2021

  1. Implement the server Connected trait for tokio UnixStream 

    This impl is needed in order to use a tokio UnixStream as the
`incoming` argument in methods like
`tonic::transport::server::Router::serve_with_incoming_shutdown`

Signed-off-by: Anthony Green <agreen@starry.com>
    @agreen17
    agreen17 committed Dec 7, 2021
    Configuration menu
    Copy the full SHA
    46650c4 View commit details
    Browse the repository at this point in the history

  2. Update uds example to use tokio UnixListenerStream 

    tokio-stream packages a UnixListenerStream that implements
futures_core::Stream. Using this cuts down on consumer boilerplate
when using UnixStreams with a tonic server.

Signed-off-by: Anthony Green <agreen@starry.com>
    @agreen17
    agreen17 committed Dec 7, 2021
    Configuration menu
    Copy the full SHA
    c1719c1 View commit details
    Browse the repository at this point in the history

  3. Add integration test for UdsConnectInfo 

    Signed-off-by: Anthony Green <agreen@starry.com>
    @agreen17
    agreen17 committed Dec 7, 2021
    Configuration menu
    Copy the full SHA
    22d1f1e View commit details
    Browse the repository at this point in the history