Standardization of error handling
Probably not the best way, but this should make the code more readable as a lot of it is just outputting error messages. This allows for centralized error messages translation.
This commit is contained in:
Softion
GitHub
parent
a376dce0c1
commit
0db686d63c
1 changed files with 109 additions and 294 deletions
299
routes/api.js
299
routes/api.js
|
|
@ -12,29 +12,27 @@ let log = require('npmlog');
|
|||
let router = new express.Router();
|
||||
let mailHelpers = require('../lib/subscription-mail-helpers');
|
||||
|
||||
router.all('/*', (req, res, next) => {
|
||||
if (!req.query.access_token) {
|
||||
res.status(403);
|
||||
const handleErrorResponse = (res, log, err, code = 500, message = false) => {
|
||||
if (typeof err != 'undefined')
|
||||
log.error('API', err);
|
||||
res.status(code);
|
||||
return res.json({
|
||||
error: 'Missing access_token',
|
||||
error: message || err.message || err,
|
||||
data: []
|
||||
});
|
||||
}
|
||||
|
||||
router.all('/*', (req, res, next) => {
|
||||
if (!req.query.access_token) {
|
||||
return handleErrorResponse(res, log, false, 403, 'Missing access_token');
|
||||
}
|
||||
|
||||
users.findByAccessToken(req.query.access_token, (err, user) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!user) {
|
||||
res.status(403);
|
||||
return res.json({
|
||||
error: 'Invalid or expired access_token',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 403, 'Invalid or expired access_token');
|
||||
}
|
||||
next();
|
||||
});
|
||||
|
|
@ -48,35 +46,17 @@ router.post('/subscribe/:listId', (req, res) => {
|
|||
});
|
||||
lists.getByCid(req.params.listId, (err, list) => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 403, 'Invalid or expired access_token');
|
||||
}
|
||||
if (!list) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Selected listId not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Selected listId not found');
|
||||
}
|
||||
if (!input.EMAIL) {
|
||||
res.status(400);
|
||||
return res.json({
|
||||
error: 'Missing EMAIL',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 400, 'Missing EMAIL');
|
||||
}
|
||||
tools.validateEmail(input.EMAIL, false, err => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(400);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err, 400);
|
||||
}
|
||||
|
||||
let subscription = {
|
||||
|
|
@ -132,22 +112,12 @@ router.post('/subscribe/:listId', (req, res) => {
|
|||
|
||||
confirmations.addConfirmation(list.id, 'subscribe', req.ip, data, (err, confirmCid) => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
|
||||
mailHelpers.sendConfirmSubscription(list, input.EMAIL, confirmCid, subscription, (err) => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
|
||||
res.status(200);
|
||||
|
|
@ -161,12 +131,7 @@ router.post('/subscribe/:listId', (req, res) => {
|
|||
} else {
|
||||
subscriptions.insert(list.id, meta, subscription, (err, response) => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -188,51 +153,26 @@ router.post('/unsubscribe/:listId', (req, res) => {
|
|||
});
|
||||
lists.getByCid(req.params.listId, (err, list) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!list) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Selected listId not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Selected listId not found');
|
||||
}
|
||||
if (!input.EMAIL) {
|
||||
res.status(400);
|
||||
return res.json({
|
||||
error: 'Missing EMAIL',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 400, 'Missing EMAIL');
|
||||
}
|
||||
|
||||
subscriptions.getByEmail(list.id, input.EMAIL, (err, subscription) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
|
||||
if (!subscription) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Subscription with given email not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Subscription with given email not found');
|
||||
}
|
||||
|
||||
subscriptions.changeStatus(list.id, subscription.id, false, subscriptions.Status.UNSUBSCRIBED, (err, found) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -253,55 +193,27 @@ router.post('/delete/:listId', (req, res) => {
|
|||
});
|
||||
lists.getByCid(req.params.listId, (err, list) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!list) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Selected listId not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Selected listId not found');
|
||||
}
|
||||
if (!input.EMAIL) {
|
||||
res.status(400);
|
||||
return res.json({
|
||||
error: 'Missing EMAIL',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 400, 'Missing EMAIL');
|
||||
}
|
||||
subscriptions.getByEmail(list.id, input.EMAIL, (err, subscription) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!subscription) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Subscription not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Subscription not found');
|
||||
}
|
||||
subscriptions.delete(list.id, subscription.cid, (err, subscription) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!subscription) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Subscription not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Subscription not found');
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -321,19 +233,11 @@ router.get('/subscriptions/:listId', (req, res) => {
|
|||
|
||||
lists.getByCid(req.params.listId, (err, list) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
subscriptions.list(list.id, start, limit, (err, rows, total) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -351,11 +255,7 @@ router.get('/subscriptions/:listId', (req, res) => {
|
|||
router.get('/lists', (req, res) => {
|
||||
lists.quicklist((err, lists) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -367,10 +267,7 @@ router.get('/lists', (req, res) => {
|
|||
router.get('/list/:id', (req, res) => {
|
||||
lists.get(req.params.id, (err, list) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -382,11 +279,7 @@ router.get('/list/:id', (req, res) => {
|
|||
router.get('/lists/:email', (req, res) => {
|
||||
lists.getListsWithEmail(req.params.email, (err, lists) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -402,19 +295,10 @@ router.post('/field/:listId', (req, res) => {
|
|||
});
|
||||
lists.getByCid(req.params.listId, (err, list) => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!list) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Selected listId not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Selected listId not found');
|
||||
}
|
||||
|
||||
let field = {
|
||||
|
|
@ -429,11 +313,7 @@ router.post('/field/:listId', (req, res) => {
|
|||
|
||||
fields.create(list.id, field, (err, id, tag) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -452,19 +332,11 @@ router.post('/blacklist/add', (req, res) => {
|
|||
input[(key || '').toString().trim().toUpperCase()] = (req.body[key] || '').toString().trim();
|
||||
});
|
||||
if (!(input.EMAIL) || (input.EMAIL === '')) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: 'EMAIL argument are required',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
blacklist.add(input.EMAIL, (err) =>{
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -479,19 +351,11 @@ router.post('/blacklist/delete', (req, res) => {
|
|||
input[(key || '').toString().trim().toUpperCase()] = (req.body[key] || '').toString().trim();
|
||||
});
|
||||
if (!(input.EMAIL) || (input.EMAIL === '')) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: 'EMAIL argument are required',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 500, 'EMAIL argument are required');
|
||||
}
|
||||
blacklist.delete(input.EMAIL, (err) =>{
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -507,11 +371,7 @@ router.get('/blacklist/get', (req, res) => {
|
|||
|
||||
blacklist.get(start, limit, search, (err, data, total) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
@ -531,92 +391,47 @@ router.post('/changeemail/:listId', (req, res) => {
|
|||
input[(key || '').toString().trim().toUpperCase()] = (req.body[key] || '').toString().trim();
|
||||
});
|
||||
if (!(input.EMAILOLD) || (input.EMAILOLD === '')) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: 'EMAILOLD argument is required',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 500, 'EMAILOLD argument is required');
|
||||
}
|
||||
if (!(input.EMAILNEW) || (input.EMAILNEW === '')) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: 'EMAILNEW argument is required',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 500, 'EMAILNEW argument is required');
|
||||
}
|
||||
lists.getByCid(req.params.listId, (err, list) => {
|
||||
if (err) {
|
||||
log.error('API', err);
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (!list) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Selected listId not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Selected listId not found');
|
||||
}
|
||||
blacklist.isblacklisted(input.EMAILNEW, (err, blacklisted) =>{
|
||||
blacklist.isblacklisted(input.EMAILNEW, (err, blacklisted) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
if (blacklisted) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: 'New email is blacklisted',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 500, 'New email is blacklisted');
|
||||
}
|
||||
|
||||
subscriptions.getByEmail(list.id, input.EMAILOLD, (err, subscription) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
|
||||
if (!subscription) {
|
||||
res.status(404);
|
||||
return res.json({
|
||||
error: 'Subscription with given old email not found',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 404, 'Subscription with given old email not found');
|
||||
}
|
||||
|
||||
subscriptions.updateAddressCheck(list, subscription.cid, input.EMAILNEW, null, (err, old, valid) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
|
||||
if (!valid) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: 'New email not valid',
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, false, 500, 'New email not valid');
|
||||
}
|
||||
|
||||
subscriptions.updateAddress(list.id, subscription.id, input.EMAILNEW, (err) => {
|
||||
if (err) {
|
||||
res.status(500);
|
||||
return res.json({
|
||||
error: err.message || err,
|
||||
data: []
|
||||
});
|
||||
return handleErrorResponse(res, log, err);
|
||||
}
|
||||
res.status(200);
|
||||
res.json({
|
||||
|
|
|
|||
Loading…
Reference in a new issue