search

su-its / ams-backend

:briefcase: (This repository is no longer maintained) The backend server of our Access-management-system.
MIT License
0 stars 0 forks source link

SSEのコネクションをたぶん正しく切れていない #51

Closed h-takeyeah closed 3 years ago

h-takeyeah commented 3 years ago

https://github.com/su-its/ams-backend-nodejs/blob/6b6a66313c371e84ab71887d57855d5ad153ba03/app/controllers/sseController.ts#L32-L38

ここのコードで外しているリスナーはSSEとは関係ないnodeの標準モジュールeventsのEventEmitterにひっついているリスナーです．これはこれでいいのですが，同じタイミングでSSEのコネクションを開いているクライアントにend()なりを送ってあげてストリームの終了を教えてあげる必要があります．それが今できていないというのが問題点です．

h-takeyeah commented 3 years ago

参考 https://github.com/su-its/ams-backend-nodejs/pull/43#discussion_r592034401