Skip to content

Commit

Permalink
fix(web): fix compilation (#670)
Browse files Browse the repository at this point in the history
Make tonic::body::empty_body public and use it instead of the old and non existant anymore BoxBody::empty()
  • Loading branch information
gmorer committed Jun 2, 2021
1 parent 9cc14b7 commit e199387
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
6 changes: 3 additions & 3 deletions tonic-web/tonic-web/src/service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ use std::task::{Context, Poll};

use http::{header, HeaderMap, HeaderValue, Method, Request, Response, StatusCode, Version};
use hyper::Body;
use tonic::body::BoxBody;
use tonic::body::{BoxBody, empty_body};
use tonic::transport::NamedService;
use tower_service::Service;
use tracing::{debug, trace};
Expand Down Expand Up @@ -65,7 +65,7 @@ where
fn no_content(&self, headers: HeaderMap) -> BoxFuture<S::Response, S::Error> {
let mut res = Response::builder()
.status(StatusCode::NO_CONTENT)
.body(BoxBody::empty())
.body(empty_body())
.unwrap();

res.headers_mut().extend(headers);
Expand All @@ -77,7 +77,7 @@ where
Box::pin(async move {
Ok(Response::builder()
.status(status)
.body(BoxBody::empty())
.body(empty_body())
.unwrap())
})
}
Expand Down
3 changes: 2 additions & 1 deletion tonic/src/body.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ pub type BoxBody = http_body::combinators::BoxBody<bytes::Bytes, crate::Status>;

// this also exists in `crate::codegen` but we need it here since `codegen` has
// `#[cfg(feature = "codegen")]`.
pub(crate) fn empty_body() -> BoxBody {
/// Create an empty `BoxBody`
pub fn empty_body() -> BoxBody {
http_body::Empty::new().map_err(|err| match err {}).boxed()
}

0 comments on commit e199387

Please sign in to comment.