Skip to content

Commit a09c4c2

Browse files
committed
Merge pull request #2 from nielsbjerg/master
Fixed hanging connection on delete.
2 parents 55c4e32 + aff53d4 commit a09c4c2

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

server/server.js

+10-1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ var db = {};
77
var template400 = '<h1>Error 400: Post syntax incorrect.</h1>\n';
88
var template404 = '<h1>404 - Not found</h1>\n';
99
var template406 = '<h1>Error 406: Not Acceptable. Package name and url must be unique.</h1>\n';
10+
var template503 = '<h1>Error 503: Database error. Please try again</h1>\n';
1011
exports.template400 = template400;
1112
exports.template404 = template404;
1213
exports.template406 = template406;
@@ -111,5 +112,13 @@ app.post('/packages', function(req, res){
111112

112113
app.delete('/packages/:name', function(req, res){
113114
var name = req.params.name;
114-
db.run('DELETE FROM packages WHERE name = $name', {$name: name});
115+
db.run('DELETE FROM packages WHERE name = $name', {$name: name}, function(err) {
116+
if(err) {
117+
res.statusCode = 503;
118+
res.send(template503);
119+
} else {
120+
res.statusCode = 204;
121+
res.send("");
122+
}
123+
});
115124
});

0 commit comments

Comments
 (0)