Add support for tcp keepalive through socket2#981
Conversation
Not fully implemented, probably should be using a custom structure for `TcpKeepalive` to avoid exposing `socket2` publicly. In that case, should we have the same feature flag as the `socket2` library? Or should we allow each setting and just warn when one is not available for the given platform ?
5109f94 to
d278529
Compare
|
Are there other things we might want to tweak using If this lands in std::net eventually, we can make the feature flags a no-op, but it'd be nice to not have many many flags in that case. |
|
I think EDIT: |
Not fully implemented, probably should be using a custom structure for
TcpKeepaliveto avoid exposingsocket2publicly.In that case, should we have the same feature flag as the
socket2library? Or should we allow each setting and just warn when one is not available for the given platform ?Fixes #547