sentry - `/cfg - add content-type for application/json

elpiel · elpiel · commit 1ea3cefd5b52 · 2019-11-07T07:41:29.000+02:00
diff --git a/sentry/src/lib.rs b/sentry/src/lib.rs
@@ -11,13 +11,17 @@ pub mod routes {
     pub mod channel;
     pub mod cfg {
         use crate::ResponseError;
+        use hyper::header::CONTENT_TYPE;
         use hyper::{Body, Response};
         use primitives::Config;
 
         pub fn return_config(config: &Config) -> Result<Response<Body>, ResponseError> {
             let config_str = serde_json::to_string(config)?;
 
-            Ok(Response::builder().body(Body::from(config_str)).unwrap())
+            Ok(Response::builder()
+                .header(CONTENT_TYPE, "application/json")
+                .body(Body::from(config_str))
+                .expect("Creating a response should never fail"))
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -11,13 +11,17 @@ pub mod routes {`
`11`	`11`	`pub mod channel;`
`12`	`12`	`pub mod cfg {`
`13`	`13`	`use crate::ResponseError;`
	`14`	`+ use hyper::header::CONTENT_TYPE;`
`14`	`15`	`use hyper::{Body, Response};`
`15`	`16`	`use primitives::Config;`
`16`	`17`
`17`	`18`	`pub fn return_config(config: &Config) -> Result<Response<Body>, ResponseError> {`
`18`	`19`	`let config_str = serde_json::to_string(config)?;`
`19`	`20`
`20`		`- Ok(Response::builder().body(Body::from(config_str)).unwrap())`
	`21`	`+ Ok(Response::builder()`
	`22`	`+ .header(CONTENT_TYPE, "application/json")`
	`23`	`+ .body(Body::from(config_str))`
	`24`	`+ .expect("Creating a response should never fail"))`
`21`	`25`	`}`
`22`	`26`	`}`
`23`	`27`	`}`