Skip to content

Commit

Permalink
fix(headers): make ConnectionHeader unicase
Browse files Browse the repository at this point in the history
Make ConnectionHeader case-insensitive since HTTP headers are
case-insensitive
  • Loading branch information
cyderize committed Jan 24, 2015
1 parent 65c7018 commit e06e7d9
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/header/common/connection.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ use header::{Header, HeaderFormat};
use std::fmt;
use std::str::FromStr;
use header::parsing::{from_comma_delimited, fmt_comma_delimited};
use unicase::UniCase;

pub use self::ConnectionOption::{KeepAlive, Close, ConnectionHeader};

Expand All @@ -26,15 +27,15 @@ pub enum ConnectionOption {
// TODO: it would be nice if these "Strings" could be stronger types, since
// they are supposed to relate to other Header fields (which we have strong
// types for).
ConnectionHeader(String),
ConnectionHeader(UniCase<String>),
}

impl FromStr for ConnectionOption {
fn from_str(s: &str) -> Option<ConnectionOption> {
match s {
"keep-alive" => Some(KeepAlive),
"close" => Some(Close),
s => Some(ConnectionHeader(s.to_string()))
s => Some(ConnectionHeader(UniCase(s.to_string())))
}
}
}
Expand All @@ -44,7 +45,7 @@ impl fmt::Display for ConnectionOption {
write!(fmt, "{}", match *self {
KeepAlive => "keep-alive",
Close => "close",
ConnectionHeader(ref s) => s.as_slice()
ConnectionHeader(UniCase(ref s)) => s.as_slice()
})
}
}
Expand Down

0 comments on commit e06e7d9

Please sign in to comment.