Skip to content

Commit

Permalink
Refactor to use ServoUrl::is_secure_scheme
Browse files Browse the repository at this point in the history
  • Loading branch information
Rohit Burra committed Dec 25, 2016
1 parent 9123fe0 commit 49abf57
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions components/net/cookie_storage.rs
Expand Up @@ -38,7 +38,7 @@ impl CookieStorage {
let cookies = self.cookies_map.entry(domain).or_insert(vec![]);

// https://www.ietf.org/id/draft-ietf-httpbis-cookie-alone-01.txt Step 2
if !cookie.cookie.secure && url.scheme() != "https" && url.scheme() != "wss" {
if !cookie.cookie.secure && !url.is_secure_scheme() {
let new_domain = cookie.cookie.domain.as_ref().unwrap();
let new_path = cookie.cookie.path.as_ref().unwrap();

Expand Down Expand Up @@ -85,7 +85,7 @@ impl CookieStorage {
// http://tools.ietf.org/html/rfc6265#section-5.3
pub fn push(&mut self, mut cookie: Cookie, url: &ServoUrl, source: CookieSource) {
// https://www.ietf.org/id/draft-ietf-httpbis-cookie-alone-01.txt Step 1
if cookie.cookie.secure && url.scheme() != "https" && url.scheme() != "wss" {
if cookie.cookie.secure && !url.is_secure_scheme() {
return;
}

Expand Down

0 comments on commit 49abf57

Please sign in to comment.