Added support for http methods when mounting routes.

lib/server.dart

Index: lib/server.dart
diff --git a/lib/server.dart b/lib/server.dart
index 124c64b6fe41ecfc548e3ed4dee4437bfb5ec4a0..373a8a27a8a864af4de75f44845b546adea4683a 100644
--- a/lib/server.dart
+++ b/lib/server.dart
@@ -51,9 +51,9 @@ typedef Future<bool> Filter(HttpRequest request);
 class Router {
   final Stream<HttpRequest> _incoming;
   
-  final Map<Pattern, StreamController> _controllers = 
-      new LinkedHashMap<Pattern, StreamController>();
-      
+  final Map<RequestPattern, StreamController> _controllers = 

[justinfagnani, 2013/04/19 19:20:51]

Make _controllers a List. See comment below.

[adambender, 2013/04/19 19:43:11]

Done.

+      new LinkedHashMap<RequestPattern, StreamController>();
+  
   final Map<Pattern, Filter> _filters = new LinkedHashMap<Pattern, Filter>();
   
   final StreamController<HttpRequest> _defaultController = 
@@ -71,12 +71,12 @@ class Router {
    * Request whose URI matches [url] are sent to the stream created by this
    * method, and not sent to any other router streams.

[justinfagnani, 2013/04/19 19:20:51]

Add docs for [method]

[adambender, 2013/04/19 19:43:11]

Done.

    */
-  Stream<HttpRequest> serve(Pattern url) {
+  Stream<HttpRequest> serve(Pattern url, {String method}) {
     var controller = new StreamController<HttpRequest>();
-    _controllers[url] = controller;
+    _controllers[new RequestPattern(url, method:method)] = controller;
     return controller.stream;
   }
-
+  
   /**
    * A [Filter] returns a [Future<bool>] that tells the router whether to apply
    * the remaining filters and send requests to the streams created by [serve].
@@ -104,9 +104,9 @@ class Router {
     }).then((_) {
       if (cont) {
         bool handled = false;
-        for (Pattern pattern in _controllers.keys) {
-          if (matchesFull(pattern, req.uri.path)) {
-            _controllers[pattern].add(req);
+        for (RequestPattern reqPattern in _controllers.keys) {

[justinfagnani, 2013/04/19 19:20:51]

if we want to be even more Darty, we can use Iterable.where (once _controllers
is a List):

var matches = _controllers.where((r) => r.matches(req))
if (!matches.isEmpty) {
  matches.first.controller.add(req);
}

[adambender, 2013/04/19 19:43:11]

Done.

+          if (reqPattern.matches(req)) {
+            _controllers[reqPattern].add(req);
             handled = true;
             break;
           }
@@ -128,3 +128,13 @@ void send404(HttpRequest req) {
   req.response.write("Not Found");
   req.response.close();
 }
+
+class RequestPattern{

[justinfagnani, 2013/04/19 19:20:51]

On the client-side I have a similar class called Route, might make sense to use
the same name. The only difference is that Route includes the handler. Here
Route could include the StreamController. I don't think the Map _controllers is
actually doing much as a Map, so it could be replaced by a list.

[adambender, 2013/04/19 19:43:11]

Done.

+  final Pattern url;
+  final String method;
+  RequestPattern(this.url, {this.method});
+  
+  bool matches(HttpRequest request){

[justinfagnani, 2013/04/19 19:20:51]

space before {

[justinfagnani, 2013/04/19 19:20:51]

you can use the "fat arrow syntax" here

[adambender, 2013/04/19 19:43:11]

Done.

[adambender, 2013/04/19 19:43:11]

Done.

+    return  matchesFull(url, request.uri.path) && (method == null || request.method.toUpperCase() == method);

[justinfagnani, 2013/04/19 19:20:51]

we're using an 80 col limit

[adambender, 2013/04/19 19:43:11]

Done.

+  }
+}