Skip to content

Commit f59118e

Browse files
author
Maitray Suthar
committed
Add helper and manage 404 page not found.
1 parent cff84f7 commit f59118e

File tree

2 files changed

+39
-0
lines changed

2 files changed

+39
-0
lines changed

app.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ var logger = require('morgan');
66
var indexRouter = require('./routes/index');
77
var usersRouter = require('./routes/users');
88

9+
var apiResponse = require('./helpers/apiResponse');
10+
911
var app = express();
1012

1113
app.use(logger('dev'));
@@ -17,4 +19,9 @@ app.use(express.static(path.join(__dirname, 'public')));
1719
app.use('/', indexRouter);
1820
app.use('/users', usersRouter);
1921

22+
// throw 404 if URL not found
23+
app.get("*", function(req, res) {
24+
return apiResponse.notFoundResponse(res, 'Page not found');
25+
});
26+
2027
module.exports = app;

helpers/apiResponse.js

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
exports.successResponse = function (res, msg) {
2+
var data = {
3+
status: 1,
4+
message: msg
5+
}
6+
return res.status(200).json(data);
7+
};
8+
9+
exports.successResponseWithData = function (res, msg, data) {
10+
var data = {
11+
status: 1,
12+
message: msg,
13+
data: data
14+
}
15+
return res.status(200).json(data);
16+
}
17+
18+
exports.ErrorResponse = function (res, msg) {
19+
var data = {
20+
status: 0,
21+
message: msg,
22+
}
23+
return res.status(500).json(data);
24+
}
25+
26+
exports.notFoundResponse = function (res, msg) {
27+
var data = {
28+
status: 0,
29+
message: msg,
30+
}
31+
return res.status(404).json(data);
32+
}

0 commit comments

Comments
 (0)