Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-01-27 22:49:56 -05:00
Code Issues Activity

Added sentry logging to member lookup for link clicks

Browse source 
no ref
This commit is contained in:
Steve Larson 2024-12-10 13:56:08 -06:00
parent 0f9449137f
commit a60756b321
2 changed files with 14 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
ghost/core/core/server/services/link-tracking/LinkClickRepository.js
View file

@ -59,11 +59,19 @@ module.exports = class LinkClickRepository {
async save(linkClick) { async save(linkClick) {
let member; let member;
if (config && config.get('linkClickTrackingCacheMemberUuid')) { sentry.startSpan({name: 'Member.findOne'}, async () => {
member = await this.memoizedFindOne(linkClick.member_uuid); try {
} else { console.log(`finding member`);
member = await this.#Member.findOne({uuid: linkClick.member_uuid}); if (config && config.get('linkClickTrackingCacheMemberUuid')) {
} member = await this.memoizedFindOne(linkClick.member_uuid);
} else {
member = await this.#Member.findOne({uuid: linkClick.member_uuid});
}
} catch (error) {
sentry.captureException(error);
throw error;
}
});
if (!member) { if (!member) {
if (config.get('bulkEmail:captureLinkClickBadMemberUuid')) { if (config.get('bulkEmail:captureLinkClickBadMemberUuid')) {

1
ghost/core/core/shared/sentry.js
View file

@ -151,6 +151,7 @@ if (sentryConfig && !sentryConfig.disabled) {
tracingHandler: Sentry.Handlers.tracingHandler(), tracingHandler: Sentry.Handlers.tracingHandler(),
captureException: Sentry.captureException, captureException: Sentry.captureException,
captureMessage: Sentry.captureMessage, captureMessage: Sentry.captureMessage,
startSpan: Sentry.startSpan,
beforeSend: beforeSend, beforeSend: beforeSend,
beforeSendTransaction: beforeSendTransaction, beforeSendTransaction: beforeSendTransaction,
initQueryTracing: (knex) => { initQueryTracing: (knex) => {