# Re: \[PATCH v3\] connect, protocol: log negotiated protocol version

- From: Taylor Blau \<hidden\>
- Date: 2021-08-16 18:03:34 
- List: git
- Page: https://ratatoskr.run/git/2021/08/7899393
- Thread: https://ratatoskr.run/git/2021/08/7898213/t.md

```text
On Tue, Aug 10, 2021 at 10:20:39AM -0700, Josh Steadmon wrote:
> It is useful for performance monitoring and debugging purposes to know
> the wire protocol used for remote operations. This may differ from the
> version set in local configuration due to differences in version and/or
> configuration between the server and the client. Therefore, log the
> negotiated wire protocol version via trace2, for both clients and
> servers.
>
> Signed-off-by: Josh Steadmon <redacted>
> ---
>
> Changes in V3:
> * remove unnecessary `test` calls in the new t5705 test cases
> * remove the wire protocol version formatting function
> * log the wire protocol version enum value directly, as in V0 of this
>   series
> * avoid logging the protocol version if negotiation fails (i.e., if we
>   end up with "protocol_unknown_version")

Nicely done. This version looks ready to be picked up by my eyes.

    Reviewed-by: Taylor Blau [ref]

Thanks,
Taylor
```