Skip to content

Add gitaly session id & catfile-cache feature flag

John Cai requested to merge jc-client-gitaly-session-id into master

What does this MR do?

In gitaly!1203 (merged), gitaly added a cache to reduce the number of git processes that get spun up.

This MR adds the session id into the header so that gitaly can do caching based on it. This feature will also be gated behind a feature flag

Does this MR meet the acceptance criteria?

Conformity

Performance and testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Michael Kozono

Merge request reports