Merge pull request #55 from damianb/patch-1

Obey HTTP spec, use HTTP 404 for not-found responses
v1.18.x
Julian Lam 12 years ago
commit c41e204a58

@ -284,7 +284,7 @@ var express = require('express'),
}); });
break; break;
default : default :
res.json({}); res.json(404, { error: 'unrecognized API endpoint' });
break; break;
} }
} }
@ -300,7 +300,7 @@ var express = require('express'),
if(data) if(data)
res.send(data); res.send(data);
else else
res.send("Category doesn't exist!"); res.send(404, "Category doesn't exist!");
}); });
}); });
@ -309,7 +309,7 @@ var express = require('express'),
if(data) if(data)
res.send(data); res.send(data);
else else
res.send("Topic doesn't exist!"); res.send(404, "Topic doesn't exist!");
}); });
}); });
@ -318,7 +318,7 @@ var express = require('express'),
if(data) if(data)
res.send(data); res.send(data);
else else
res.send("Post doesn't exist!"); res.send(404, "Post doesn't exist!");
}); });
}); });
@ -352,4 +352,4 @@ var express = require('express'),
}(WebServer)); }(WebServer));
server.listen(config.port); server.listen(config.port);
global.server = server; global.server = server;

Loading…
Cancel
Save