updated API docs for delete

brianewing · Dec 28, 2011 · 22623d5 · 22623d5
1 parent b7e3bc8
commit 22623d5
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 14 deletions.
diff --git a/app.js b/app.js
@@ -132,7 +132,7 @@ myapp.put('/app', middle.authenticate, middle.authenticate_app, app.put);
 
 // Delete your nodejs app
 // curl -X DELETE -u "testuser:123" -d "appname=test" http://localhost:4001/apps
-myapp.del('/app/:appname', middle.authenticate, middle.authenticate_app, app.deleteapp);
+myapp.del('/app/:appname', middle.authenticate, middle.authenticate_app, app.delete);
 
 
 myapp.del('/gitreset/:appname', middle.authenticate, middle.authenticate_app, app.gitreset);
@@ -164,7 +164,7 @@ myapp.post('/npm', middle.authenticate, middle.authenticate_app, npm.post);
 // curl -X DELETE -u "testuser:123" -d "appname=test&domain=<domainname>" http://localhost:4001/appdomains
 var domains = require('./lib/domains');
 myapp.post('/appdomains', middle.authenticate, middle.authenticate_app, domains.post);
-myapp.del('/appdomains', middle.authenticate, middle.authenticate_app, domains.delete);
+myapp.del('/appdomains/:appname/:domain', middle.authenticate, middle.authenticate_app, domains.delete);
 myapp.get('/appdomains', middle.authenticate, domains.get);
 
 // curl -X POST -d "user=username" http://localhost:4001/reset_password

diff --git a/lib/app.js b/lib/app.js
@@ -91,7 +91,7 @@ module.exports = {
       }
     });
   },
-  deleteapp: function (req, res, next) {
+  delete: function (req, res, next) {
     var appname = req.param("appname").toLowerCase();
     // var appname = req.body.appname.toLowerCase();
 	// var appname = req.appname;

diff --git a/lib/domains.js b/lib/domains.js
@@ -49,9 +49,10 @@ module.exports = {
     }
   },
   delete: function(req, res, next) {
-    var appname = req.appname,
-        domain = req.param("domain"),
-        user = req.user,
+	var appname = req.param("appname").toLowerCase();
+    var domain = req.param("domain").toLowerCase();
+
+    var user = req.user,
         app = req.app;
 
     var gooddomain = lib.checkDomain(domain);

diff --git a/public/api.html b/public/api.html
@@ -309,15 +309,15 @@ <h1>Nodester API Documentation</h1>
                         <tr>
                            <td>&nbsp;</td>
                            <td><a class="btn disabled danger">DELETE</a></td>
-                           <td><b>/app </b>- Deletes nodejs app (requires basic auth and appname)<br />
-                              <code>curl -X DELETE -u "testuser:123" -d "appname=test" http://api.nodester.com/app</code>
+                           <td><b>/app/&lt;appname&gt; </b>- Deletes nodejs app (requires basic auth and appname)<br />
+                              <code>curl -X DELETE -u "testuser:123" http://api.nodester.com/app/&lt;appname&gt;</code>
                            </td>
                         </tr>
                         <tr>
                            <td>&nbsp;</td>
                            <td><a class="btn disabled primary">GET</a></td>
                            <td><b>/app/&lt;appname&gt;</b> - get nodejs app info (requires basic auth and appname)<br />
-                              <code>curl -u "testuser:123"  http://api.nodester.com/app/appname</code>
+                              <code>curl -u "testuser:123"  http://api.nodester.com/app/&lt;appname&gt;</code>
                            </td>
                         </tr>
                         <tr>
@@ -329,7 +329,7 @@ <h1>Nodester API Documentation</h1>
                            <td>&nbsp;</td>
                            <td><a class="btn disabled primary">GET</a></td>
                            <td><b>/applogs/&lt;appname&gt;</b> - Gets nodejs app logs (requires basic auth and appname)<br />
-                              <code>curl -u "testuser:123"  http://api.nodester.com/applogs/appname</code>
+                              <code>curl -u "testuser:123"  http://api.nodester.com/applogs/&lt;appname&gt;</code>
                            </td>
                         </tr>
                         <tr>
@@ -399,8 +399,8 @@ <h1>Nodester API Documentation</h1>
                         <tr>
                            <td>&nbsp;</td>
                            <td><a class="btn disabled danger">DELETE</a></td>
-                           <td><b>/appdomains</b> - Deletes app domain for hosting example.com (requires basic auth)<br />
-                              <code>curl -X DELETE -u "testuser:123" -d "appname=test&domain=example.com" http://api.nodester.com/appdomains</code>
+                           <td><b>/appdomains/&lt;appname&gt;/&lt;domain&gt;</b> - Deletes app domain for hosting example.com (requires basic auth)<br />
+                              <code>curl -X DELETE -u "testuser:123" http://api.nodester.com/appdomains/&lt;appname&gt;/&lt;domain&gt;</code>
                            </td>
                         </tr>
                         <tr>
@@ -437,8 +437,8 @@ <h1>Nodester API Documentation</h1>
                         <tr>
                            <td>&nbsp;</td>
                            <td><a class="btn disabled danger">DELETE</a></td>
-                           <td><b>/gitreset</b> - Resets your Nodester git repository and npm list (pass in appname)<br />
-                              <code>curl -X DELETE -u "testuser:123" -d "appname=a" http://api.nodester.com/gitreset</code>
+                           <td><b>/gitreset/&lt;appname&gt;</b> - Resets your Nodester git repository and npm list (pass in appname)<br />
+                              <code>curl -X DELETE -u "testuser:123" http://api.nodester.com/gitreset/&lt;appname&gt;</code>
                            </td>
                         </tr>
                      </table>