webdriver: Fix invalid check for scheme in proxy host. (#114)

Source-Repo: https://github.com/mozilla/webdriver-rust
Source-Revision: 12e62a7956219ca4ebb3d6161e4ce9c853e82217

committer: jgraham <james@hoppipolla.co.uk>

--HG--
extra : subtree_source : http%3A//tristan.corp.lon2.mozilla.com%3A8000
extra : subtree_revision : fffe353f879f73c03225355001d5547924fa73e8

testing/webdriver/src/capabilities.rs

@@ -203,7 +203,7 @@ impl SpecNewSessionParameters {
                             value: &Json) -> WebDriverResult<()> {
         match value.as_string() {
             Some(x) => {
-                if x.contains("::/") {
+                if x.contains("://") {
                     return Err(WebDriverError::new(
                         ErrorStatus::InvalidArgument,
                         format!("{} contains a scheme", name)));
@@ -552,6 +552,7 @@ mod tests {
         validate_host("ftpProxy", "ftp", "{}", "example.org").unwrap();
         validate_host("ftpProxy", "ftp", "{}", "::1").unwrap();
         assert!(validate_host("ftpProxy", "ftp", "{}", "ftp://example.org").is_err());
+        assert!(validate_host("ftpProxy", "ftp", "{}", "ftp:/example.org").is_err());
         assert!(validate_host("ftpProxy", "ftp", "{}", "example.org/foo").is_err());
         assert!(validate_host("ftpProxy", "ftp", "{}", "example.org#bar").is_err());
         assert!(validate_host("ftpProxy", "ftp", "{}", "example.org?bar=baz").is_err());