Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NestJS Bull appears to be utilizing Upstash Redis unnecessarily. #2029

Closed
1 task done
dsmabulage opened this issue Mar 10, 2024 · 1 comment
Closed
1 task done

NestJS Bull appears to be utilizing Upstash Redis unnecessarily. #2029

dsmabulage opened this issue Mar 10, 2024 · 1 comment
Labels
bug Something isn't working needs triage

Comments

@dsmabulage
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current behavior

NestJS Bull utilises Upstash Redis unnecessarily, even when not using the queues. This behaviour results in unnecessary resource consumption and increased costs.

Minimum reproduction code

Steps to reproduce

BullModule.registerQueue({
      name: QUEUE_NAME,
      defaultJobOptions: {
        delay: 5000,
        removeOnComplete: true,
        removeOnFail: true,
        attempts: 3,
      },
    }),

Bull module

BullModule.forRootAsync({
     imports: [ConfigModule],
     inject: [ConfigService],
     useFactory(configService: ConfigService) {
          return {
           redis: {
             host: `${configService.get('REDIS_HOST')}:${configService.get(
               'REDIS_PORT',
             )}
               `,
             port: configService.get('REDIS_PORT'),
             password: configService.get('REDIS_PASSWORD'),
             tls: {
               rejectUnauthorized: true,
               host: configService.get('REDIS_HOST'),
               port: configService.get<number>('REDIS_PORT'),
             },
           },
         };      },
   })
   ```

### Expected behavior

NestJS Bull should only utilize Upstash Redis when actively using the queues for job processing.

### Package version

10.1.0

### Bull version

4.12.2

### NestJS version

10.0.0

### Node.js version

20.11.0

### In which operating systems have you tested?

- [ ] macOS
- [X] Windows
- [ ] Linux

### Other

_No response_
@dsmabulage dsmabulage added bug Something isn't working needs triage labels Mar 10, 2024
@kamilmysliwiec
Copy link
Member

This package doesn't use Upstash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage
Projects
None yet
Development

No branches or pull requests

2 participants