I may be mistaken but should it not send the negotiated interval back in `tune_ok`?

Also we should preserve the existence of a `heartbeat` attribute for backwards compatibility, I guess the `heartbeat_interval` attribute could simply be renamed `heartbeat`, otherwise we would have to add it as an alias and deprecate it.