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

gRPC client stream not working when interceptor is registered in NestJS #13818

Open
3 of 15 tasks
hersentino opened this issue Jul 23, 2024 · 1 comment
Open
3 of 15 tasks
Labels
needs triage This issue has not been looked into

Comments

@hersentino
Copy link

hersentino commented Jul 23, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current behavior

With a NestJS gRPC server that uses a interceptor, client stream calls are not working. If no interceptor is used, client streams work flawlessly. The same behavior occurs for bidirectional streams, but there are no bugs for server streams

Minimum reproduction code

https://github.com/hersentino/nestjs-grpc-interceptor

Steps to reproduce

  1. npm i
  2. npm start => run gRPC server
  3. npm run client => run gRPC client that makes a client stream to the server => call not working
  4. Comment out the gRPC interceptor on line 28 inside src/main.ts
  5. Re-run steps 2 and 3 => call working fine

Expected behavior

To have the gRPC client stream working when an interceptor is registered

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

No response

Packages versions

Nest CLI Version : 10.4.2 

[Nest Platform Information]
platform-express version : 10.3.10
microservices version    : 10.3.10
schematics version       : 10.1.2
testing version          : 10.3.10
common version           : 10.3.10
core version             : 10.3.10
cli version              : 10.4.2

Node.js version

v20.13.1

In which operating systems have you tested?

  • macOS
  • Windows
  • Linux

Other

No response

@hersentino hersentino added the needs triage This issue has not been looked into label Jul 23, 2024
@Kelvin-ui-hub
Copy link

Hi, can i work on this issue?

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

2 participants