rivernews / slack-middleware-server

This server act as a middleware to communicate with Slack API.
1 stars 1 forks source link

Memory leak in DEV env #88

Closed rivernews closed 4 years ago

rivernews commented 4 years ago

npm run watch may cause unclosed redis client. We observed a idle redis client connection more than 200 after developing in DEV a couple few times. The number 2XX is the amount after s3 job finished, finalized and no work in progress.

If we still want to have hot-reload, as a workaround, we may just want to use npm run fast-dev, just make sure everytime we save file to refresh, wait till the resources get clean up before saving file again.

Or we can just not use hot-reload. Always run nom run dev, abort and re-run it manually once finish making code changes and ready to test.

rivernews commented 4 years ago

Since there's no obvious fix for this, we'll move these instructions to the README.