Skip to content

Commit e76b590

Browse files
committed
feat(http): Add optional serialization of common types via serde.
This is behind a Cargo feature to avoid forcing downstream users to depend on `serde`. It is needed for Servo IPC to work.
1 parent 623824d commit e76b590

File tree

6 files changed

+131
-0
lines changed

6 files changed

+131
-0
lines changed

Cargo.toml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,16 @@ optional = true
3636
version = "0.4"
3737
default-features = false
3838

39+
[dependencies.serde]
40+
version = "*"
41+
optional = true
42+
3943
[dev-dependencies]
4044
env_logger = "*"
4145

4246
[features]
4347
default = ["ssl"]
4448
ssl = ["openssl", "cookie/secure"]
49+
serde-serialization = ["serde"]
4550
nightly = []
51+

src/header/common/content_type.rs

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
use mime::Mime;
22

3+
#[cfg(feature = "serde-serialization")]
4+
use header::Header;
5+
#[cfg(feature = "serde-serialization")]
6+
use serde::{Deserialize, Deserializer, Serialize, Serializer};
7+
38
header! {
49
#[doc="`Content-Type` header, defined in"]
510
#[doc="[RFC7231](http://tools.ietf.org/html/rfc7231#section-3.1.1.5)"]
@@ -94,4 +99,20 @@ impl ContentType {
9499
}
95100
}
96101

102+
#[cfg(feature = "serde-serialization")]
103+
impl Serialize for ContentType {
104+
fn serialize<S>(&self, serializer: &mut S) -> Result<(), S::Error> where S: Serializer {
105+
format!("{}", self).serialize(serializer)
106+
}
107+
}
108+
109+
#[cfg(feature = "serde-serialization")]
110+
impl Deserialize for ContentType {
111+
fn deserialize<D>(deserializer: &mut D) -> Result<ContentType, D::Error>
112+
where D: Deserializer {
113+
let string_representation: String = try!(Deserialize::deserialize(deserializer));
114+
Ok(Header::parse_header(&[string_representation.into_bytes()]).unwrap())
115+
}
116+
}
117+
97118
bench_header!(bench, ContentType, { vec![b"application/json; charset=utf-8".to_vec()] });

src/header/mod.rs

Lines changed: 66 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,13 @@ use unicase::UniCase;
9292

9393
use self::internals::Item;
9494

95+
#[cfg(feature = "serde-serialization")]
96+
use serde::{Deserialize, Deserializer, Serialize, Serializer};
97+
#[cfg(feature = "serde-serialization")]
98+
use serde::de;
99+
#[cfg(feature = "serde-serialization")]
100+
use serde::ser;
101+
95102
pub use self::shared::*;
96103
pub use self::common::*;
97104

@@ -322,6 +329,65 @@ impl fmt::Debug for Headers {
322329
}
323330
}
324331

332+
#[cfg(feature = "serde-serialization")]
333+
impl Serialize for Headers {
334+
fn serialize<S>(&self, serializer: &mut S) -> Result<(), S::Error> where S: Serializer {
335+
struct HeadersVisitor<'a> {
336+
iter: HeadersItems<'a>,
337+
len: usize,
338+
}
339+
340+
impl<'a> ser::MapVisitor for HeadersVisitor<'a> {
341+
fn visit<S>(&mut self, serializer: &mut S) -> Result<Option<()>, S::Error>
342+
where S: Serializer {
343+
match self.iter.next() {
344+
Some(header_item) => {
345+
try!(serializer.visit_map_elt(header_item.name(),
346+
header_item.value_string()));
347+
Ok(Some(()))
348+
}
349+
None => Ok(None),
350+
}
351+
}
352+
353+
fn len(&self) -> Option<usize> {
354+
Some(self.len)
355+
}
356+
}
357+
358+
serializer.visit_map(HeadersVisitor {
359+
iter: self.iter(),
360+
len: self.len(),
361+
})
362+
}
363+
}
364+
365+
#[cfg(feature = "serde-serialization")]
366+
impl Deserialize for Headers {
367+
fn deserialize<D>(deserializer: &mut D) -> Result<Headers, D::Error> where D: Deserializer {
368+
struct HeadersVisitor;
369+
370+
impl de::Visitor for HeadersVisitor {
371+
type Value = Headers;
372+
373+
fn visit_map<V>(&mut self, mut visitor: V) -> Result<Headers, V::Error>
374+
where V: de::MapVisitor {
375+
let mut result = Headers::new();
376+
while let Some((key, value)) = try!(visitor.visit()) {
377+
let (key, value): (String, String) = (key, value);
378+
result.set_raw(key, vec![value.into_bytes()]);
379+
}
380+
try!(visitor.end());
381+
Ok(result)
382+
}
383+
}
384+
385+
let result = Headers::new();
386+
try!(deserializer.visit_map(HeadersVisitor));
387+
Ok(result)
388+
}
389+
}
390+
325391
/// An `Iterator` over the fields in a `Headers` map.
326392
pub struct HeadersItems<'a> {
327393
inner: Iter<'a, HeaderName, Item>

src/http/mod.rs

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ use header::Headers;
77
use version::HttpVersion;
88
use version::HttpVersion::{Http10, Http11};
99

10+
#[cfg(feature = "serde-serialization")]
11+
use serde::{Deserialize, Deserializer, Serialize, Serializer};
12+
1013
pub use self::message::{HttpMessage, RequestHead, ResponseHead, Protocol};
1114

1215
pub mod h1;
@@ -17,6 +20,21 @@ pub mod message;
1720
#[derive(Clone, PartialEq, Debug)]
1821
pub struct RawStatus(pub u16, pub Cow<'static, str>);
1922

23+
#[cfg(feature = "serde-serialization")]
24+
impl Serialize for RawStatus {
25+
fn serialize<S>(&self, serializer: &mut S) -> Result<(), S::Error> where S: Serializer {
26+
(self.0, self.1.clone().into_owned()).serialize(serializer)
27+
}
28+
}
29+
30+
#[cfg(feature = "serde-serialization")]
31+
impl Deserialize for RawStatus {
32+
fn deserialize<D>(deserializer: &mut D) -> Result<RawStatus, D::Error> where D: Deserializer {
33+
let representation: (u16, String) = try!(Deserialize::deserialize(deserializer));
34+
Ok(RawStatus(representation.0, Cow::Owned(representation.1)))
35+
}
36+
}
37+
2038
/// Checks if a connection should be kept alive.
2139
#[inline]
2240
pub fn should_keep_alive(version: HttpVersion, headers: &Headers) -> bool {

src/lib.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,8 @@ extern crate time;
133133
extern crate url;
134134
#[cfg(feature = "openssl")]
135135
extern crate openssl;
136+
#[cfg(feature = "serde-serialization")]
137+
extern crate serde;
136138
extern crate cookie;
137139
extern crate unicase;
138140
extern crate httparse;

src/method.rs

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ use error::Error;
77
use self::Method::{Options, Get, Post, Put, Delete, Head, Trace, Connect, Patch,
88
Extension};
99

10+
#[cfg(feature = "serde-serialization")]
11+
use serde::{Deserialize, Deserializer, Serialize, Serializer};
12+
1013
/// The Request Method (VERB)
1114
///
1215
/// Currently includes 8 variants representing the 8 methods defined in
@@ -125,6 +128,21 @@ impl fmt::Display for Method {
125128
}
126129
}
127130

131+
#[cfg(feature = "serde-serialization")]
132+
impl Serialize for Method {
133+
fn serialize<S>(&self, serializer: &mut S) -> Result<(), S::Error> where S: Serializer {
134+
format!("{}", self).serialize(serializer)
135+
}
136+
}
137+
138+
#[cfg(feature = "serde-serialization")]
139+
impl Deserialize for Method {
140+
fn deserialize<D>(deserializer: &mut D) -> Result<Method, D::Error> where D: Deserializer {
141+
let string_representation: String = try!(Deserialize::deserialize(deserializer));
142+
Ok(FromStr::from_str(&string_representation[..]).unwrap())
143+
}
144+
}
145+
128146
#[cfg(test)]
129147
mod tests {
130148
use std::collections::HashMap;

0 commit comments

Comments
 (0)