Updated uri scheme parsing according to spec.

http/src/main/scala/spinoco/protocol/http/HttpScheme.scala

@@ -1,27 +1,44 @@
 package spinoco.protocol.http
 
-import scodec.Codec
+import scodec.{Codec, Err}
 import codec.helper._
-import spinoco.protocol.common.util._
 
+/**
+  * A generic protocol for Uri.
+  *
+  * @param tpe  The type of the protocol.
+  */
+case class Scheme private[http](tpe: String)
+
+object Scheme {
+
+  def parseScheme(toParse: String): Option[Scheme] = {
+    toParse.headOption.flatMap { firstChar =>
+      if (!firstChar.isLetter) None
+      else if (toParse.tail.forall(c => c.isLetterOrDigit || c == '+' || c == '-' || c == '.')) {
+        Some(new Scheme(toParse.toLowerCase))
+      } else None
+    }
+  }
 
-object HttpScheme extends Enumeration {
+  val codec: Codec[Scheme] = {
+    import scodec.Attempt._
+    trimmedAsciiToken.exmap(
+      s => fromOption(parseScheme(s), Err(s"The specified scheme is not valid according to RFC 3986: $s"))
+      , s => successful(s.tpe)
+    )
+  }
 
-  val HTTP = Value("http")
+}
 
-  val HTTPS = Value("https")
+object HttpScheme {
 
-  val WS = Value("ws")
+  val HTTP = Scheme("http")
 
-  val WSS = Value("wss")
+  val HTTPS = Scheme("https")
 
+  val WS = Scheme("ws")
 
-  val codec: Codec[HttpScheme.Value] = {
-    import scodec.Attempt._
-    trimmedAsciiToken.exmap(
-      s => attempt(HttpScheme.withName(s.toLowerCase))
-      , s => successful(s.toString)
-    )
-  }
+  val WSS = Scheme("wss")
 
 }

http/src/main/scala/spinoco/protocol/http/Uri.scala

@@ -17,7 +17,7 @@ import scala.annotation.tailrec
   * All values are decoded (no % escaping)
   */
 sealed case class Uri(
-  scheme: HttpScheme.Value
+  scheme: Scheme
   , host: HostPort
   , path: Uri.Path
   , query: Uri.Query
@@ -96,7 +96,7 @@ object Uri {
       )
     }
 
-    (terminated(HttpScheme.codec, Terminator.constantString1("://")) ~ hostPathQueryCodec)
+    (terminated(Scheme.codec, Terminator.constantString1("://")) ~ hostPathQueryCodec)
     .xmap(
       { case (scheme, (host, path, query)) => Uri(scheme, host, path, query) }
       , uri => (uri.scheme, (uri.host, uri.path, uri.query))

http/src/test/scala/spinoco/protocol/http/SchemeSpec.scala

@@ -0,0 +1,24 @@
+package spinoco.protocol.http
+
+import org.scalacheck.Properties
+import org.scalacheck.Prop._
+
+object SchemeSpec extends Properties("Schema"){
+
+  property("parse.valid") = protect{
+    Scheme.parseScheme("http-1+3.two") ?= Some(Scheme("http-1+3.two"))
+  }
+
+  property("parse.valid.upper-case") = protect{
+    Scheme.parseScheme("http-1+3.TWO") ?= Some(Scheme("http-1+3.two"))
+  }
+
+  property("parse.invalid.starts-digit") = protect{
+    Scheme.parseScheme("1http-1+3.two") ?= None
+  }
+
+  property("parse.invalid.non-valid-char") = protect{
+    Scheme.parseScheme("http-1:3.two") ?= None
+  }
+
+}