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

sse close event not trigger #13956

Closed
2 of 15 tasks
wenghongtian opened this issue Sep 10, 2024 · 2 comments
Closed
2 of 15 tasks

sse close event not trigger #13956

wenghongtian opened this issue Sep 10, 2024 · 2 comments
Labels
needs triage This issue has not been looked into

Comments

@wenghongtian
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current behavior

sse close event not trigger

Minimum reproduction code

https://github.com/wenghongtian/nest-sse

Steps to reproduce

  1. pnpm i
  2. pnpm start
  3. call 'http://localhost:3004/api/workflow/sse/application/run' with sse and post method
  4. close the connect
  5. req.on('close', () => { console.log('request closed'); }); not trigger

Expected behavior

When the socket is closed, 'close' event can trigger

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

[System Information]
OS Version : macOS 23.1.0
NodeJS Version : v20.12.2
PNPM Version : 8.14.3

[Nest CLI]
Nest CLI Version : 10.4.2

[Nest Platform Information]
platform-express version : 10.3.10
microservices version : 10.3.10
platform-ws version : 10.3.10
websockets version : 10.3.10
schematics version : 10.1.2
throttler version : 5.2.0
passport version : 10.0.3
swagger version : 7.4.0
testing version : 10.3.10
common version : 10.3.10
config version : 3.2.3
axios version : 3.0.2
core version : 10.3.10
jwt version : 10.2.0
cli version : 10.4.2

Node.js version

20.12.2

In which operating systems have you tested?

  • macOS
  • Windows
  • Linux

Other

No response

@wenghongtian wenghongtian added the needs triage This issue has not been looked into label Sep 10, 2024
@micalevisk
Copy link
Member

maybe related with this issue: #10389

@kamilmysliwiec
Copy link
Member

Duplicate #10389

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

3 participants