Skip to content

workhorse: refactor access to Gitaly Sidechannel

Vasilii Iakliushin requested to merge 472033_extract_sidechannel into master

What does this MR do and why?

Contributes to #472033 (closed)

Problem

NewConnectionWithSidechannel returns Sidechannel even if it's not needed.

Solution

Split NewConnectionWithSidechannel function into:

  • NewConnection() to get Gitaly connection
  • Sidechannel() to access Gitaly sidechannel

Merge request reports