Added comments

knowlecules · May 9, 2012 · dd1e201 · dd1e201
1 parent 2b7202c
commit dd1e201
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/rapid-rest.js b/lib/rapid-rest.js
@@ -63,9 +63,9 @@ module.exports = function() {
                         }
                     });
                     req.on('end', function () {
-                        //TODO: Switch contentType
+                        //TODO: Switch contentType or pass parser as a parameter
 
-                        var data = body;// JSON.parse(body);
+                        var data = JSON.parse(body);
                         fn(req, res, segments, data);
                     });
                 }
@@ -101,17 +101,21 @@ module.exports = function() {
 
                 var urlDef;
                 var urlPath = req.url;
+
+                //Optimization for URLs that have no query string
                 if(req.url.indexOf("?")){
                     urlDef =  url.parse(req.url,true);
                     urlPath = urlDef.pathname;
+                    req.query = urlDef.query;
                 }
 
+                // Check for matching pattern
                 for (var pattern in rest.matchDefs) {
                     var matchDef = rest.matchDefs[pattern];
                     var urlParams = urlPath.match(matchDef.re);
 
                     if ( urlParams !== null){
-                        // Validate required querystring parameters
+                        // Validate required querystring kvps
                         if(urlDef && matchDef.queryKeys){
                             for(var kvp in urlDef.query)  {
                                 if(!matchDef.query[kvp]){