Adding keepalive for client socket

maximkulkin / esp-homekit

Apple HomeKit accessory server library for ESP-OPEN-RTOS

MIT License

1.11k stars 170 forks source link

Adding keepalive for client socket #37

Closed overbog closed 6 years ago

overbog commented 6 years ago

Confronting myself with issue https://github.com/maximkulkin/esp-homekit/issues/35 I found out that it's not a true memory leak but just a bunch long dead connections. They will be dropped eventually by TCP stack but in low memory environments this may come too late... . Adding keepalive on client socket solved that for me.

maximkulkin commented 6 years ago

Looks good. Thank you for troubleshooting this. Just a minor comment