Skip to content

Commit d0f3867

Browse files
committed
add event bus call logging
1 parent 8f72422 commit d0f3867

File tree

2 files changed

+22
-11
lines changed

2 files changed

+22
-11
lines changed

src/events/busApi.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ module.exports = (app, logger) => {
2828
projectId: project.id,
2929
projectName: project.name,
3030
initiatorUserId: req.authUser.userId,
31-
});
31+
}, logger);
3232
});
3333

3434
/**
@@ -43,7 +43,7 @@ module.exports = (app, logger) => {
4343
projectId: updated.id,
4444
projectName: updated.name,
4545
initiatorUserId: req.authUser.userId,
46-
});
46+
}, logger);
4747
} else if (
4848
!_.isEqual(original.details, updated.details) ||
4949
!_.isEqual(original.name, updated.name) ||
@@ -53,14 +53,14 @@ module.exports = (app, logger) => {
5353
projectId: updated.id,
5454
projectName: updated.name,
5555
initiatorUserId: req.authUser.userId,
56-
});
56+
}, logger);
5757
} else if (!_.isEqual(original.bookmarks, updated.bookmarks)) {
5858
logger.debug('project bookmarks is updated');
5959
createEvent(BUS_API_EVENT.PROJECT_LINK_CREATED, {
6060
projectId: updated.id,
6161
projectName: updated.name,
6262
initiatorUserId: req.authUser.userId,
63-
});
63+
}, logger);
6464
}
6565
});
6666

@@ -93,7 +93,7 @@ module.exports = (app, logger) => {
9393
projectName: project.name,
9494
userId: member.userId,
9595
initiatorUserId: req.authUser.userId,
96-
});
96+
}, logger);
9797
}).catch(err => null); // eslint-disable-line no-unused-vars
9898
});
9999

@@ -121,7 +121,7 @@ module.exports = (app, logger) => {
121121
projectName: project.name,
122122
userId: member.userId,
123123
initiatorUserId: req.authUser.userId,
124-
});
124+
}, logger);
125125
}
126126
}).catch(err => null); // eslint-disable-line no-unused-vars
127127
});
@@ -144,7 +144,7 @@ module.exports = (app, logger) => {
144144
projectName: project.name,
145145
userId: updated.userId,
146146
initiatorUserId: req.authUser.userId,
147-
});
147+
}, logger);
148148
}
149149
}).catch(err => null); // eslint-disable-line no-unused-vars
150150
}
@@ -167,7 +167,7 @@ module.exports = (app, logger) => {
167167
projectName: project.name,
168168
fileName: attachment.filePath.replace(/^.*[\\\/]/, ''), // eslint-disable-line
169169
initiatorUserId: req.authUser.userId,
170-
});
170+
}, logger);
171171
}).catch(err => null); // eslint-disable-line no-unused-vars
172172
});
173173
};

src/services/busApi.js

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,16 +36,27 @@ function getClient() {
3636
* Any errors will be simply ignored
3737
* @param {String} type the event type, should be a dot separated fully qualitied name
3838
* @param {Object} message the message, should be a JSON object
39+
* @param {Object} logger object
3940
* @return {Promise} new event promise
4041
*/
41-
function createEvent(type, message) {
42+
function createEvent(type, message, logger) {
4243
const body = JSON.stringify(message);
44+
logger.debug(`Sending message: ${JSON.stringify(message)}`);
4345
return getClient().post('/eventbus/events', {
4446
type,
4547
message: body,
4648
})
47-
.then(resp => resp)
48-
.catch(error => Promise.resolve()); // eslint-disable-line
49+
.then((resp) => {
50+
logger.debug('Sent event to bus-api');
51+
logger.debug(`Sent event to bus-api [data]: ${resp.data}`);
52+
logger.debug(`Sent event to bus-api [status]: ${resp.status}`);
53+
})
54+
.catch((error) => {
55+
logger.debug('Error sending event to bus-api');
56+
logger.debug(`Error sending event to bus-api [message]: ${error.message}`);
57+
logger.debug(`Error sending event to bus-api [detail]: ${error.response.data.message}`);
58+
Promise.resolve(); // eslint-disable-line
59+
});
4960
}
5061

5162

0 commit comments

Comments
 (0)