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 microservices sentinels not connected. #13199

Closed
6 of 19 tasks
avisiboni opened this issue Feb 11, 2024 · 0 comments
Closed
6 of 19 tasks

NestJS microservices sentinels not connected. #13199

avisiboni opened this issue Feb 11, 2024 · 0 comments
Labels
needs triage This issue has not been looked into

Comments

@avisiboni
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current behavior

I've created a new project with NestJS

Info:
Node v20.11.0
NPM v9.1.3
NestJS v10.0.0

Minimum reproduction code

https://stackblitz.com/edit/nestjs-typescript-starter-udeeaz?file=package.json

Steps to reproduce

Here is the steps list in order to reproduce (repo link attached).

  • A new ms project based on redis transport created.
  • Redis config first point to the localhost, and it work as expected.
  • Redis config changed to our sentinel server and it that point is stop to work.

What I've tried.

  • New Redis with ioredis directly with the same configuration - works perfect, one line below with the same config, its throw WRONGPASS invalid username-password pair or user is disabled, which no makesense.
    image

Expected behavior

The Redis initiation should work just the same as manual initiation works

Package

  • I don't know. Or some 3rd-party package
  • @nestjs/common
  • @nestjs/core
  • @nestjs/microservices
  • @nestjs/platform-express
  • @nestjs/platform-fastify
  • @nestjs/platform-socket.io
  • @nestjs/platform-ws
  • @nestjs/testing
  • @nestjs/websockets
  • Other (see below)

Other package

No response

NestJS version

10.0.0

Packages versions

 "dependencies": {
    "@nestjs/axios": "^3.0.1",
    "@nestjs/cache-manager": "2.2.0",
    "@nestjs/common": "^10.0.0",
    "@nestjs/config": "^3.1.1",
    "@nestjs/core": "^10.0.0",
    "@nestjs/microservices": "^10.3.0",
    "@nestjs/platform-express": "^10.0.0",
    "axios": "^1.6.5",
    "cache-manager": "5.3.2",
    "cache-manager-ioredis-yet": "^1.2.2",
    "env-var": "^7.4.1",
    "ioredis": "^5.3.2",
    "kafkajs": "^2.2.4",
    "menora.libs.logger": "^4.8.3",
    "menora.libs.send.email": "^2.0.2",
    "node-vault": "^0.10.2",
    "redis": "^4.6.12",
    "reflect-metadata": "^0.1.13",
    "rxjs": "^7.8.1"
  },
  "devDependencies": {
    "@nestjs/cli": "^10.0.0",
    "@nestjs/schematics": "^10.0.0",
    "@nestjs/testing": "^10.0.0",
    "@types/express": "^4.17.17",
    "@types/jest": "^29.5.2",
    "@types/node": "^20.3.1",
    "@types/supertest": "^6.0.0",
    "@typescript-eslint/eslint-plugin": "^6.0.0",
    "@typescript-eslint/parser": "^6.0.0",
    "eslint": "^8.42.0",
    "eslint-config-prettier": "^9.0.0",
    "eslint-plugin-prettier": "^5.0.0",
    "jest": "^29.5.0",
    "prettier": "^3.0.0",
    "source-map-support": "^0.5.21",
    "supertest": "^6.3.3",
    "ts-jest": "^29.1.0",
    "ts-loader": "^9.4.3",
    "ts-node": "^10.9.1",
    "tsconfig-paths": "^4.2.0",
    "typescript": "^5.1.3"
  },

Node.js version

20.11.0

In which operating systems have you tested?

  • macOS
  • Windows
  • Linux

Other

No response

@avisiboni avisiboni added the needs triage This issue has not been looked into label Feb 11, 2024
@avisiboni avisiboni closed this as not planned Won't fix, can't repro, duplicate, stale Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage This issue has not been looked into
Projects
None yet
Development

No branches or pull requests

1 participant