Skip to content

Commit 917ffba

Browse files
VibhorCodecianGuptaVibhorCodecianGupta
VibhorCodecianGupta
authored and
VibhorCodecianGupta
committed
raven for exception handling
1 parent a9b7d67 commit 917ffba

12 files changed

+154
-83
lines changed

‎.gitignore

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@
22
node_modules/
33
.DS_Store/
44
public_html/bower_components/
5-
/secrets.json
5+
secrets.json
66
npm-debug.log
77
newrelic_agent.log

‎auth/passport.js

+4-3
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ const models = require('../db/models');
44
const config = require('../config');
55
const uid = require('uid2')
66
const bearerStrategy = require('./strategies/bearerStrategy');
7-
7+
const Raven = require('raven');
88

99
passport.use('oneauth', new oneauthStrategy({
1010
authorizationURL: 'https://account.codingblocks.com/oauth/authorize',
@@ -34,7 +34,8 @@ passport.use('oneauth', new oneauthStrategy({
3434
).then(function (authtokenObject) {
3535
return done(null, authtokenObject[0].get())
3636
}).catch(function (err) {
37-
console.log(err);
37+
Raven.captureException(err)
38+
return done(err, false);
3839
})
3940
}
4041
else {
@@ -63,4 +64,4 @@ passport.deserializeUser(function (user, done) {
6364
});
6465
passport.use('bearer', bearerStrategy);
6566

66-
module.exports = passport;
67+
module.exports = passport;

‎auth/strategies/bearerStrategy.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
const BearerStrategy = require('passport-http-bearer').Strategy;
22
const models = require('./../../db/models');
3-
3+
const Raven = require('raven');
44

55
module.exports = new BearerStrategy(function (token, done) {
66

@@ -22,6 +22,7 @@ module.exports = new BearerStrategy(function (token, done) {
2222
return done(null, false, {message: 'Could not authorize'});
2323
}
2424
}).catch(function (err) {
25+
Raven.captureException(err)
2526
return done(err, false);
2627
});
2728

‎config.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ switch (config.DEPLOY_CONFIG) {
1818
break;
1919

2020
case 'localhost':
21-
config.secrets = require('./secrets-sample')
21+
config.secrets = require('./secrets')
2222
config.db = config.secrets.DATABASE
2323
config.clientId = config.secrets.ONEAUTH.CLIENT_ID;
2424
config.clientSecret = config.secrets.ONEAUTH.CLIENT_SECRET;
@@ -45,4 +45,4 @@ config.SENTRY_DSN = process.env.SENTRY_DSN || config.secrets.SENTRY_DSN
4545
config.NEW_RELIC_LICENSE_KEY = process.env.NEW_RELIC_LICENSE_KEY || config.secrets.NEW_RELIC_LICENSE_KEY
4646

4747

48-
exports = module.exports = config
48+
exports = module.exports = config

‎db/actions/batches.js

+20-10
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
*
33
* Created by tech4GT on 8/25/17.
44
*/
5+
const Raven = require('raven')
56
const models = require('../models')
67
module.exports = {
78

@@ -23,7 +24,8 @@ module.exports = {
2324
}).then(function (data) {
2425
done(null, data)
2526
}).catch(function (err) {
26-
if (err) done(err);
27+
Raven.captureException(err)
28+
if (err) done(err)
2729
});
2830
},
2931
getAll: function (conditions, done) {
@@ -35,7 +37,8 @@ module.exports = {
3537

3638
done(null, data)
3739
}).catch(function (err) {
38-
if (err) done(err);
40+
Raven.captureException(err)
41+
if (err) done(err)
3942
});
4043
},
4144
search: function (id, done) {
@@ -50,7 +53,8 @@ module.exports = {
5053
}).then(function (data) {
5154
done(null, data);
5255
}).catch(function (err) {
53-
if (err) done(err);
56+
Raven.captureException(err)
57+
if (err) done(err)
5458
});
5559
},
5660
edit: function (id, obj, done) {
@@ -65,10 +69,12 @@ module.exports = {
6569
data.update(obj).then(function (resData) {
6670
done(null, resData);
6771
}).catch(function (err) {
68-
if (err) done(err);
72+
Raven.captureException(err)
73+
if (err) done(err)
6974
})
7075
}).catch(function (err) {
71-
if (err) done(err);
76+
Raven.captureException(err)
77+
if (err) done(err)
7278
});
7379
},
7480
archiveBatch: function (id, done) {
@@ -85,10 +91,12 @@ module.exports = {
8591
}).then(function (resData) {
8692
done(null, resData);
8793
}).catch(function (err) {
88-
if (err) done(err);
94+
Raven.captureException(err)
95+
if (err) done(err)
8996
})
9097
}).catch(function (err) {
91-
if (err) done(err);
98+
Raven.captureException(err)
99+
if (err) done(err)
92100
});
93101
},
94102
deleteBatch: function (id, done) {
@@ -99,7 +107,8 @@ module.exports = {
99107
}).then(function (data) {
100108
done(null, data)
101109
}).catch(function (err) {
102-
if (err) done(err);
110+
Raven.captureException(err)
111+
if (err) done(err)
103112
});
104113
}
105114
,
@@ -112,7 +121,8 @@ module.exports = {
112121
}).then(function (data) {
113122
done(null, data)
114123
}).catch(function (err) {
115-
if (err) done(err);
124+
Raven.captureException(err)
125+
if (err) done(err)
116126
});
117127
}
118-
}
128+
}

‎db/actions/centres.js

+20-13
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
* Created by abhishekyadav on 25/08/17.
33
*/
44
const models = require('../models')
5+
const Raven = require('raven')
56

67
module.exports = {
78
createNew: function (name, head, phone, done) {
@@ -12,14 +13,16 @@ module.exports = {
1213
}).then(function (data) {
1314
done(null, data)
1415
}).catch(function (err) {
15-
if (err) done(err);
16+
Raven.captureException(err)
17+
if (err) done(err)
1618
});
1719
},
1820
getAll: function (done) {
1921
models.Centres.findAll({order: ['id']}).then(function (data) {
2022
done(null, data)
2123
}).catch(function (err) {
22-
if (err) done(err);
24+
Raven.captureException(err)
25+
if (err) done(err)
2326
});
2427
},
2528
search: function (id, done) {
@@ -30,7 +33,8 @@ module.exports = {
3033
}).then(function (data) {
3134
done(null, data)
3235
}).catch(function (err) {
33-
if (err) done(err);
36+
Raven.captureException(err)
37+
if (err) done(err)
3438
});
3539
},
3640
edit: function (id, obj, done) {
@@ -45,10 +49,12 @@ module.exports = {
4549
data.update(obj).then(function (resData) {
4650
done(null, resData);
4751
}).catch(function (err) {
48-
if (err) done(err);
52+
Raven.captureException(err)
53+
if (err) done(err)
4954
})
5055
}).catch(function (err) {
51-
if (err) done(err);
56+
Raven.captureException(err)
57+
if (err) done(err)
5258
});
5359
},
5460
deleteCentre: function (id, done) {
@@ -59,7 +65,8 @@ module.exports = {
5965
}).then(function (data) {
6066
done(null, data)
6167
}).catch(function (err) {
62-
if (err) done(err);
68+
Raven.captureException(err)
69+
if (err) done(err)
6370
});
6471
},
6572
getBatches: function (id, done) {
@@ -72,8 +79,8 @@ module.exports = {
7279
}).then(function (data) {
7380
done(null, data);
7481
}).catch(function (err) {
75-
if (err)
76-
done(err);
82+
Raven.captureException(err)
83+
if (err) done(err)
7784
})
7885
},
7986
getActiveBatches: function (id, done) {
@@ -87,8 +94,8 @@ module.exports = {
8794
}).then(function (data) {
8895
done(null, data);
8996
}).catch(function (err) {
90-
if (err)
91-
done(err);
97+
Raven.captureException(err)
98+
if (err) done(err)
9299
})
93100
},
94101
getRooms: function (id, done) {
@@ -100,8 +107,8 @@ module.exports = {
100107
}).then(function (data) {
101108
done(null, data);
102109
}).catch(function (err) {
103-
if (err)
104-
done(err);
110+
Raven.captureException(err)
111+
if (err) done(err)
105112
})
106113
}
107-
}
114+
}

‎db/actions/courses.js

+18-11
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/**
22
* Created by tech4GT on 8/25/17.
33
*/
4-
4+
const Raven = require('raven')
55
const models = require('../models')
66

77
module.exports = {
@@ -14,14 +14,16 @@ module.exports = {
1414
}).then(function (data) {
1515
done(null, data)
1616
}).catch(function (err) {
17-
if (err) done(err);
17+
Raven.captureException(err)
18+
if (err) done(err)
1819
});
1920
},
2021
getAll: function (done) {
2122
models.Courses.findAll({order: ['id']}).then(function (data) {
2223
done(null, data)
2324
}).catch(function (err) {
24-
if (err) done(err);
25+
Raven.captureException(err)
26+
if (err) done(err)
2527
});
2628
},
2729
search: function (id, done) {
@@ -32,7 +34,8 @@ module.exports = {
3234
}).then(function (data) {
3335
done(null, data)
3436
}).catch(function (err) {
35-
if (err) done(err);
37+
Raven.captureException(err)
38+
if (err) done(err)
3639
});
3740
},
3841
edit: function (id, obj, done) {
@@ -47,10 +50,12 @@ module.exports = {
4750
data.update(obj).then(function (resData) {
4851
done(null, resData);
4952
}).catch(function (err) {
50-
if (err) done(err);
53+
Raven.captureException(err)
54+
if (err) done(err)
5155
})
5256
}).catch(function (err) {
53-
if (err) done(err);
57+
Raven.captureException(err)
58+
if (err) done(err)
5459
});
5560
},
5661
deleteCourse: function (id, done) {
@@ -61,7 +66,8 @@ module.exports = {
6166
}).then(function (data) {
6267
done(null, data)
6368
}).catch(function (err) {
64-
if (err) throw err;
69+
Raven.captureException(err)
70+
if (err) done(err)
6571
});
6672
},
6773
getlectures: function (id, done) {
@@ -86,7 +92,8 @@ module.exports = {
8692
}
8793
done(arr);
8894
}).catch(function (err) {
89-
if (err) done(err);
95+
Raven.captureException(err)
96+
if (err) done(err)
9097
});
9198
},
9299
getBatches: function (id, done) {
@@ -98,8 +105,8 @@ module.exports = {
98105
}).then(function (data) {
99106
done(null, data);
100107
}).catch(function (err) {
101-
if (err)
102-
done(err);
108+
Raven.captureException(err)
109+
if (err) done(err)
103110
})
104111
}
105-
}
112+
}

0 commit comments

Comments
 (0)