Skip to content

Commit fc7d515

Browse files
Updated logger instances to use new logger module.
1 parent 7088136 commit fc7d515

File tree

15 files changed

+88
-112
lines changed

15 files changed

+88
-112
lines changed

src/auth/redis-auth.guard.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { Injectable, UnauthorizedException, CanActivate, ExecutionContext, createParamDecorator } from '@nestjs/common';
2-
import { LogService } from '../common/log/log.service';
2+
import { LoggerService } from '../common/logger/logger.service';
33
import { RedisService } from '../common/redis/redis.service';
44
import { RedisServer } from '../common/redis/redis.decorator';
55
import { Observable } from 'rxjs';
@@ -10,7 +10,7 @@ import { DatabaseService } from '../common/database/database.service';
1010
export class RedisAuthGuard implements CanActivate {
1111
constructor(
1212
@RedisServer('AUTH') private redis: RedisService,
13-
private readonly logService: LogService,
13+
private readonly logger: LoggerService,
1414
private readonly databaseService: DatabaseService
1515
) {}
1616

@@ -42,7 +42,7 @@ export class RedisAuthGuard implements CanActivate {
4242
if (!user) throw new UnauthorizedException();
4343
request.user = user;
4444
} catch (e) {
45-
console.log('error', e);
45+
this.logger.error('error', e);
4646
throw new UnauthorizedException();
4747
}
4848

src/cloudflare-kv/cloudflare-kv.service.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { Injectable, OnModuleInit } from '@nestjs/common';
22
import { ConfigService } from './../common/configs/config.service';
3-
import { LogService } from './../common/log/log.service';
3+
import { LoggerService } from '../common/logger/logger.service';
44
import { KvNamespaces } from './models/kv-namespaces.enum';
55

66
@Injectable()
@@ -10,7 +10,7 @@ export class CloudflareKvService implements OnModuleInit {
1010

1111
constructor(
1212
private readonly configService: ConfigService,
13-
private readonly logService: LogService
13+
private readonly logger: LoggerService
1414
) {}
1515

1616
onModuleInit() {
@@ -31,7 +31,7 @@ export class CloudflareKvService implements OnModuleInit {
3131

3232
return response;
3333
} catch (e) {
34-
this.logService.error('Failed to update workers KV', {
34+
this.logger.error('Failed to update workers KV', {
3535
errorMessage: e.message,
3636
});
3737
}
@@ -50,7 +50,7 @@ export class CloudflareKvService implements OnModuleInit {
5050

5151
return response.json();
5252
} catch (e) {
53-
this.logService.error('Failed to update workers KV', {
53+
this.logger.error('Failed to update workers KV', {
5454
errorMessage: e.message,
5555
});
5656
}

src/common/common.module.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,13 @@ import { Global, Module } from '@nestjs/common';
22
import { DatabaseModule } from './database/database.module';
33
//import { QueueModule } from './queue/queue.module';
44
import { ConfigModule } from './configs/config.module';
5-
import { LogModule } from './log/log.module';
5+
//import { LogModule } from './log/log.module';
66
import { TraceModule } from './trace/trace.module';
77
import { RedisModule } from './redis/redis.module';
88

99
@Global()
1010
@Module({
11-
imports: [DatabaseModule, ConfigModule, LogModule, TraceModule, RedisModule.forRoot()],
12-
exports: [DatabaseModule, ConfigModule, LogModule, TraceModule, RedisModule],
11+
imports: [DatabaseModule, ConfigModule, TraceModule, RedisModule.forRoot()],
12+
exports: [DatabaseModule, ConfigModule, TraceModule, RedisModule],
1313
})
1414
export class CommonModule {}

src/common/database/database.service.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
import { Injectable, OnModuleDestroy, OnModuleInit } from '@nestjs/common';
2-
import { MeterProvider } from '@opentelemetry/sdk-metrics';
3-
import { Counter } from '@opentelemetry/api-metrics';
2+
//import { MeterProvider } from '@opentelemetry/sdk-metrics';
3+
//import { Counter } from '@opentelemetry/api-metrics';
44
import { Prisma, PrismaClient, PrismaPromise } from '@prisma/client';
55
import { ConfigService } from '../configs/config.service';
6-
import { LogService } from '../log/log.service';
6+
import { LoggerService } from '../logger/logger.service';
77
import { TraceService } from '../trace/trace.service';
8-
import { logger } from '../../logger/logger';
8+
//import { logger } from '../../logger/logger';
99
// Alternatively, if the above does not work
1010
// import { PrismaClient } from '../../node_modules/@prisma/client/dist/index';
1111
// No replacement needed, just remove this line
@@ -18,7 +18,7 @@ export class DatabaseService extends PrismaClient implements OnModuleInit, OnMod
1818
//private readonly pendingQueriesCounter: UpDownCounter;
1919

2020
constructor(
21-
private readonly logService: LogService,
21+
private readonly logger: LoggerService,
2222
private readonly configService: ConfigService,
2323
private readonly traceService: TraceService //private readonly meter,
2424
) {
@@ -35,7 +35,7 @@ export class DatabaseService extends PrismaClient implements OnModuleInit, OnMod
3535
// },
3636
// ]
3737
});
38-
logger.info(`Using Prisma v${Prisma.prismaVersion.client} ${configService.get('DATABASE_URL')}`);
38+
this.logger.info(`Using Prisma v${Prisma.prismaVersion.client} ${configService.get('DATABASE_URL')}`);
3939

4040
/* this.meter = meterProvider.getMeter('example-exporter-collector');
4141
@@ -68,7 +68,7 @@ export class DatabaseService extends PrismaClient implements OnModuleInit, OnMod
6868

6969
await (this as any).$connect();
7070

71-
this.logService.info(
71+
this.logger.info(
7272
`Connected to ${prismaEngine.activeProvider} instance using Prisma v${prismaEngine.clientVersion} ${this.configService.get('DATABASE_URL')}`
7373
);
7474

@@ -92,6 +92,6 @@ export class DatabaseService extends PrismaClient implements OnModuleInit, OnMod
9292
async onModuleDestroy(): Promise<void> {
9393
await super.$disconnect();
9494

95-
this.logService.debug('Disconnected from Postgres instance');
95+
this.logger.debug('Disconnected from Postgres instance');
9696
}
9797
}

src/common/database/mongo-database.service.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { Injectable, OnModuleDestroy, OnModuleInit } from '@nestjs/common';
22
import { MeterProvider } from '@opentelemetry/sdk-metrics';
33
import { Counter } from '@opentelemetry/api';
44
import { ConfigService } from './../configs/config.service';
5-
import { LogService } from './../log/log.service';
5+
import { LoggerService } from '../logger/logger.service';
66
import { TraceService } from './../trace/trace.service';
77
//export { default as DatabaseModel, Prisma as DatabaseClause, Prisma, PrismaClient, PrismaPromise } from '@prisma/client';
88
import { PrismaClient as MongoPrismaClient, Prisma, PrismaPromise } from '@prisma/client';
@@ -13,7 +13,7 @@ export class MongoDatabaseService extends MongoPrismaClient implements OnModuleI
1313
//private readonly pendingQueriesCounter: UpDownCounter;
1414

1515
constructor(
16-
private readonly logService: LogService,
16+
private readonly logger: LoggerService,
1717
private readonly configService: ConfigService,
1818
private readonly traceService: TraceService //private readonly meter,
1919
) {
@@ -52,7 +52,7 @@ export class MongoDatabaseService extends MongoPrismaClient implements OnModuleI
5252

5353
await this.$connect();
5454

55-
this.logService.info(
55+
this.logger.info(
5656
`Connected to ${prismaEngine.activeProvider} instance using Prisma v${prismaEngine.clientVersion}`
5757
);
5858

@@ -87,6 +87,6 @@ export class MongoDatabaseService extends MongoPrismaClient implements OnModuleI
8787
async onModuleDestroy(): Promise<void> {
8888
await this.$disconnect();
8989

90-
this.logService.debug('Disconnected from Mongo instance');
90+
this.logger.debug('Disconnected from Mongo instance');
9191
}
9292
}

src/common/queue/queue.service.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { Counter } from '@opentelemetry/api';
33
//import { Meter } from '@opentelemetry/sdk-metrics-base';
44
import { ConnectionOptions, Job as BullJob, JobsOptions, Queue, RedisClient } from 'bullmq';
55
import { ConfigService } from '../configs/config.service';
6-
import { LogService } from '../log/log.service';
6+
import { LoggerService } from '../logger/logger.service';
77
import { TraceService } from '../trace/trace.service';
88
import { JobNames } from './models/job-names.enum';
99
import { QueueNames } from './models/queue-names.enum';
@@ -20,7 +20,7 @@ export class QueueService implements OnModuleInit, OnModuleDestroy {
2020

2121
constructor(
2222
private readonly configService: ConfigService,
23-
private readonly logService: LogService,
23+
private readonly logger: LoggerService,
2424
private readonly traceService: TraceService //private readonly meter: Meter,
2525
) {
2626
/*this.queuedJobsCounter = this.meter.createCounter(
@@ -74,7 +74,7 @@ export class QueueService implements OnModuleInit, OnModuleDestroy {
7474
})
7575
);
7676

77-
this.logService.info('Connected to Redis');
77+
this.logger.info('Connected to Redis');
7878
} catch (error) {
7979
this.onConnectionError(error);
8080
}
@@ -95,7 +95,7 @@ export class QueueService implements OnModuleInit, OnModuleDestroy {
9595
await this.bullQueues[queue].disconnect();
9696
}
9797

98-
this.logService.debug('Disconnected from Redis instance');
98+
this.logger.debug('Disconnected from Redis instance');
9999
}
100100

101101
/**
@@ -106,7 +106,7 @@ export class QueueService implements OnModuleInit, OnModuleDestroy {
106106
* @memberof QueueService
107107
*/
108108
private onConnectionError(error: Error): void {
109-
this.logService.error('Connection error', {
109+
this.logger.error('Connection error', {
110110
errorMessage: error.message,
111111
errorStack: error.stack,
112112
});
@@ -124,7 +124,7 @@ export class QueueService implements OnModuleInit, OnModuleDestroy {
124124
});
125125

126126
span.setAttributes({ id: job.id, name: job.name });
127-
this.logService.info('Queued job', { id: job.id, name: job.name });
127+
this.logger.info('Queued job', { id: job.id, name: job.name });
128128
span.end();
129129

130130
return job;
@@ -154,7 +154,7 @@ export class QueueService implements OnModuleInit, OnModuleDestroy {
154154
const jobs = await this.bullQueues[queueName].addBulk(jobsPayload);
155155

156156
const logJobs = jobs.map((job) => ({ id: job.id, name: job.name }));
157-
this.logService.info('Queued jobs', {
157+
this.logger.info('Queued jobs', {
158158
jobs: logJobs.length,
159159
jobsSamples: JSON.stringify(logJobs.slice(0, 100)),
160160
});

src/common/redis/redis.providers.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ function redisFactory(redis: RedisService, prefix: string, cfg: ConfigService):
2626
throw new Error(`[REDIS] No server definition for prefix "${prefix}" in REDIS_SERVERS`);
2727
}
2828

29-
console.log(`[REDIS] Connecting to ${server.host}:${server.port}`);
29+
//console.log(`[REDIS] Connecting to ${server.host}:${server.port}`);
3030

3131
const redisOptions = {
3232
keepAlive: 30000,
@@ -155,4 +155,4 @@ export const redisProviders: Provider[] = [
155155
];
156156

157157
/* For visibility while booting */
158-
console.log('redisProviders', redisProviders);
158+
//console.log('redisProviders', redisProviders);

src/jobs/job.controller.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ import { RedditService } from './reddit.service';
3030
import { NotificationsService } from './notifications.service';
3131
import { PaginatedResponse } from '../common/models/paginated-response';
3232
import { CreateJobDto as JobDto } from './dto/create-job.dto';
33+
import { LoggerService } from '../common/logger/logger.service';
3334

3435
class PaginatedJobResponse extends PaginatedResponse(JobDto) {}
3536

@@ -46,13 +47,12 @@ class PaginatedJobResponse extends PaginatedResponse(JobDto) {}
4647
@ApiTags('jobs')
4748
@Controller('jobs')
4849
export class JobController {
49-
private readonly logger = new Logger(JobController.name);
50-
5150
constructor(
5251
private readonly jobService: JobService,
5352
private readonly web3CareerService: Web3CareerService,
5453
private readonly redditService: RedditService,
55-
private readonly notificationsService: NotificationsService
54+
private readonly notificationsService: NotificationsService,
55+
private readonly logger: LoggerService
5656
) {}
5757

5858
/**
@@ -91,7 +91,7 @@ export class JobController {
9191
},
9292
};
9393

94-
this.logger.log(
94+
this.logger.info(
9595
`Job fetch summary: ${summary.totalFetched} total fetched (${summary.reddit.fetched} Reddit, ${summary.web3career.fetched} Web3Career), ${summary.totalAdded} added, ${summary.totalSkipped} skipped (existing)`
9696
);
9797

src/jobs/reddit.service.ts

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,27 @@
1-
import { Injectable, Logger } from '@nestjs/common';
1+
import { Injectable, Logger as Logger2 } from '@nestjs/common';
22
import { JobService } from './job.service';
33
import { NotificationsService } from '../notifications/notifications.service';
44
import { NotificationPayload } from '../notifications/interfaces/notification-payload.interface';
55
import { DatabaseService } from '../common/database/database.service';
6+
import { LoggerService } from '../common/logger/logger.service';
67

78
@Injectable()
89
export class RedditService {
9-
private readonly logger = new Logger(RedditService.name);
10+
private readonly logger2 = new Logger2(RedditService.name);
1011

1112
constructor(
1213
private readonly jobService: JobService,
1314
private readonly notificationsService: NotificationsService,
15+
private readonly logger: LoggerService,
1416
private readonly db: DatabaseService
1517
) {}
1618
async fetchRedditPosts(subreddits: string[]): Promise<any[]> {
1719
const axios = await import('axios');
1820
const allPosts = [];
19-
this.logger.log(`Fetching posts from ${subreddits.length} subreddit(s)`);
21+
this.logger.info(`Fetching posts from ${subreddits.length} subreddit(s)`);
2022
for (const subreddit of subreddits) {
2123
try {
22-
this.logger.log(`Fetching /r/${subreddit}...`);
24+
this.logger.info(`Fetching /r/${subreddit}...`);
2325
const response = await axios.default.get(`https://www.reddit.com/r/${subreddit}/new.json?limit=10`, {
2426
timeout: 5000,
2527
});
@@ -34,7 +36,7 @@ export class RedditService {
3436
upvotes: child.data.ups,
3537
downvotes: child.data.downs,
3638
}));
37-
this.logger.log(`Found ${subredditPosts.length} post(s) in /r/${subreddit}`);
39+
this.logger.info(`Found ${subredditPosts.length} post(s) in /r/${subreddit}`);
3840
allPosts.push(...subredditPosts);
3941
} catch (error: any) {
4042
this.logger.error(`Error fetching /r/${subreddit}:`, error.message);
@@ -46,7 +48,7 @@ export class RedditService {
4648
async storeRedditJobPosts(posts: any[]): Promise<any[]> {
4749
const newJobs = [];
4850
let skippedCount = 0;
49-
this.logger.log(`Processing ${posts.length} Reddit post(s) for jobs table`);
51+
this.logger.info(`Processing ${posts.length} Reddit post(s) for jobs table`);
5052
const subscriptions = await this.db.subscription.findMany();
5153
for (const post of posts) {
5254
try {
@@ -90,14 +92,14 @@ export class RedditService {
9092
this.notificationsService.sendNotification(sub, notificationPayload)
9193
);
9294
await Promise.all(notificationPromises);
93-
this.logger.log(`Stored new job [${post.url}] from /u/${post.author}`);
95+
this.logger.info(`Stored new job [${post.url}] from /u/${post.author}`);
9496
newJobs.push(upsertedJob);
9597
} catch (error: any) {
9698
this.logger.error(`Error processing Reddit post ${post.url}:`, error);
9799
}
98100
}
99101

100-
this.logger.log(
102+
this.logger.info(
101103
`Reddit jobs processed: ${posts.length} fetched, ${skippedCount} skipped (existing), ${newJobs.length} added`
102104
);
103105

src/jobs/web3career.service.ts

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
1-
import { Injectable, Logger } from '@nestjs/common';
1+
import { Injectable } from '@nestjs/common';
2+
import { LoggerService } from '../common/logger/logger.service';
23
import { JobService } from './job.service';
34

45
@Injectable()
56
export class Web3CareerService {
6-
private readonly logger = new Logger(Web3CareerService.name);
7-
8-
constructor(private readonly jobService: JobService) {}
7+
constructor(
8+
private readonly jobService: JobService,
9+
private readonly logger: LoggerService
10+
) {}
911

1012
private get web3CareerApiUrl(): string {
1113
const token = process.env.WEB3CAREER_API_TOKEN || 'Rg9PrsGP96Z2GB6T9tNZ1AzHzriQEwxa';
@@ -18,7 +20,7 @@ export class Web3CareerService {
1820
if (!response.ok) throw new Error(`Web3Career API request failed with status: ${response.status}`);
1921
const data = await response.json();
2022
const jobsArray = Array.isArray(data[2]) ? data[2] : [];
21-
this.logger.log(`Web3Career jobs fetched: ${jobsArray.length}`);
23+
this.logger.info(`Web3Career jobs fetched: ${jobsArray.length}`);
2224
return jobsArray;
2325
} catch (error: any) {
2426
this.logger.error(`Error fetching Web3Career jobs: ${error.message}`);
@@ -64,7 +66,7 @@ export class Web3CareerService {
6466
}
6567
}
6668

67-
this.logger.log(
69+
this.logger.info(
6870
`Web3Career jobs processed: ${jobs.length} fetched, ${skippedCount} skipped (existing), ${newJobs.length} added`
6971
);
7072

0 commit comments

Comments
 (0)