Skip to content

Commit d5b977a

Browse files
authored
Merge pull request #191 from hackmcgill/develop
Bug fixes, email templates, "change password" route.
2 parents 19ffede + e7045d3 commit d5b977a

30 files changed

Lines changed: 4949 additions & 65 deletions

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,5 +64,7 @@ gcp_creds.json
6464
#certbot
6565
certbot/
6666

67+
assets/.DS_Store
68+
6769
#secrets
6870
secret.yaml

app.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,16 @@ app.use((err, req, res, next) => {
9595
const status = (err.status) ? err.status : 500;
9696
const message = (err.message) ? err.message : "Internal Server Error";
9797
//Only show bad error when we're not in deployment
98-
const errorContents = (err.error) ? err.error : (process.env.NODE_ENV !== "deployment") ? err : {};
98+
let errorContents;
99+
if (status === 500 && Services.env.isProduction) {
100+
errorContents = {};
101+
} else if (err.error) {
102+
errorContents = err.error;
103+
} else if (err.data) {
104+
errorContents = err.data;
105+
} else {
106+
errorContents = err;
107+
}
99108
res.status(status).json({
100109
message: message,
101110
data: errorContents

0 commit comments

Comments
 (0)