fix(services): correct log messages and error handling in event services
- Capitalize 'Event' in debug logs for consistency - Fix syntax error in EventWatchSchedule debug message (removed trailing backtick) - Upgrade subscription error logging from debug to error level for better visibility - Standardize event URL format in match detail logs These changes improve logging accuracy and ensure critical subscription errors are not missed.
This commit is contained in:
parent
77530d4c65
commit
3cd47a1b4d
@ -34,7 +34,7 @@ export class EventWatchSchedule implements Schedule {
|
|||||||
const oldPlayers = await redis.get(key).then(s => s ? JSON.parse(s) as Player[] : []);
|
const oldPlayers = await redis.get(key).then(s => s ? JSON.parse(s) as Player[] : []);
|
||||||
await redis.set(key, JSON.stringify(event.players));
|
await redis.set(key, JSON.stringify(event.players));
|
||||||
if (oldPlayers.length === 0) {
|
if (oldPlayers.length === 0) {
|
||||||
console.debug('event first time watch, skip compare. event: %s - %s', event.eventId, event.title);
|
console.debug('Event first time watch, skip compare. event: %s - %s', event.eventId, event.title);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
const playerMap = new Map<string, Player>(oldPlayers.map(e => [e.uid, e]));
|
const playerMap = new Map<string, Player>(oldPlayers.map(e => [e.uid, e]));
|
||||||
@ -45,7 +45,7 @@ export class EventWatchSchedule implements Schedule {
|
|||||||
const removedPlayers = diff.deletedList.map(e => playerMap.get(e) as Player);
|
const removedPlayers = diff.deletedList.map(e => playerMap.get(e) as Player);
|
||||||
const addedPlayers = diff.newMemberList.map(e => playerMap.get(e) as Player);
|
const addedPlayers = diff.newMemberList.map(e => playerMap.get(e) as Player);
|
||||||
if (!removedPlayers.length && !addedPlayers.length) {
|
if (!removedPlayers.length && !addedPlayers.length) {
|
||||||
console.debug('No change detected in event: %s - %s`,', event.eventId, event.title);
|
console.debug('No change detected in event: %s - %s,', event.eventId, event.title);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
const title = `${event.title} 人员变动`;
|
const title = `${event.title} 人员变动`;
|
||||||
|
|||||||
@ -7,7 +7,7 @@ export class EventSubscribeService {
|
|||||||
const success = await prisma.eventSubs.create({ data: { logto_uid: user, event_id: event }})
|
const success = await prisma.eventSubs.create({ data: { logto_uid: user, event_id: event }})
|
||||||
.then(() => true)
|
.then(() => true)
|
||||||
.catch(e => {
|
.catch(e => {
|
||||||
console.debug('Subscribe user: %s, event: %s, error: %s', user, event, e);
|
console.error('Subscribe user: %s, event: %s, error: %s', user, event, e);
|
||||||
return false;
|
return false;
|
||||||
});
|
});
|
||||||
return success;
|
return success;
|
||||||
@ -67,7 +67,7 @@ export class EventSubscribeService {
|
|||||||
const details: EventDetail[] = [];
|
const details: EventDetail[] = [];
|
||||||
for (const e of beforeEvents) {
|
for (const e of beforeEvents) {
|
||||||
const result = await KaiqiuService.getMatchDetail(e.eventId, true)
|
const result = await KaiqiuService.getMatchDetail(e.eventId, true)
|
||||||
console.debug('Get match detail: %s - %s, url: %s', e.title, e.eventId, e.url)
|
console.debug('Get match detail: %s - %s, url: %s', e.title, e.eventId, `https://tt.ksr.la/event/${e.eventId}`)
|
||||||
details.push(result);
|
details.push(result);
|
||||||
}
|
}
|
||||||
return details;
|
return details;
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user