Age | Commit message (Collapse) | Author |
|
|
|
Add changelog for security release
See merge request gitlab/gitaly!23
|
|
|
|
This reverts commit 664cb41f50cf9a343d59dfb8398d058c60ca2aab.
|
|
|
|
[Gitaly 1.20.1] Loosen regex for exception sanitization
See merge request gitlab/gitaly!20
|
|
Remove unused variable
See merge request gitlab-org/gitaly!1200
|
|
|
|
Bring back a custom dialer for Gitaly Ruby
Closes gitlab-ce#53473 and #1447
See merge request gitlab-org/gitaly!1072
|
|
This is a second fix for the GRPC bug related to using unix sockets
alongside unix sockets. Previously we fixed incoming clients. This
change fixes the connection between Gitaly and Gitaly-Ruby, which
always relies on unix sockets, and which currently fail when http
proxies are setup.
A (too) long history of this change:
* Unix sockets have never been officially supported by gRPC
* Originally unix sockets didn't work at all, and you had to use a
dialer
* Attempted to upgrade from gRPC 1.9 to 1.16 and found that our dialer
broke connectivity:
https://gitlab.com/gitlab-org/gitaly/merge_requests/972
* Isolated the problem to this commit:
https://github.com/grpc/grpc-go/commit/90dca43332f6cc944c37e16f32a82c41639e7705
* Fixed the problem by switching to the default dialer for unix: URLs
* (note that it's still not officially supported, but without
integration tests with proxy configurations, it seems to work)
* In #1447, discovered that the default dialer breaks if one has http
proxies configured (there was a also a second failure in
https://gitlab.com/gitlab-org/gitaly/merge_requests/1032 which
involved clients talking to Gitaly)
* This change partially reverts
https://gitlab.com/gitlab-org/gitaly/merge_requests/972 while keeping
the fix for the original connectivity issue
|
|
Remove Codecov as service from the cover job
Closes #1479
See merge request gitlab-org/gitaly!1067
|
|
Pass thru proxy
Closes #1472
See merge request gitlab-org/gitaly!1064
|
|
|
|
HA Design Document
See merge request gitlab-org/gitaly!1058
|
|
|
|
Explicitly favour testify over t.Testing functions
See merge request gitlab-org/gitaly!1070
|
|
Throughout the codebase almost always testify's subpackages are used.
But this soft requirement wasn't documented. Now it is.
The language doesn't require it, as there might be edge cases that
require or assert don't cover.
Discussed in:
https://gitlab.com/gitlab-org/gitaly/merge_requests/1064#note_139006382
|
|
|
|
Use empty tree if initial commit
See merge request gitlab-org/gitaly!1075
|
|
|
|
GRPC does not officially support unix sockets, although recent
versions do superficially support it through the `unix:` prefix.
Unfortunately this support fails when http proxy configurations
are used.
This change adds tests to test this situation
|
|
|
|
into 'master'
Return blank checksum for git repositories with only invalid refs
Closes #1480
See merge request gitlab-org/gitaly!1065
|
|
Use chunker in GetRawChanges
See merge request gitlab-org/gitaly!1043
|
|
|
|
|
|
Statically link jaeger into Gitaly by default
See merge request gitlab-org/gitaly!1063
|
|
Prior to this change, Codecov would post a note after a pipeline to tell
where coverage was gained and lost. However, these numbers never seemed
correct and changed quite suddenly so I've grown to not trust them.
This change removed Codecov; post merge I'll remove the token so it
keeps working until that time.
Closes https://gitlab.com/gitlab-org/gitaly/issues/1479
|
|
Use chunker in FindCommits
See merge request gitlab-org/gitaly!1059
|
|
|
|
If the git repository contains only refs that do not match the ones
in the whitelist the checksum variable is not initialized throwing
an error: "CalculateChecksum: invalid memory address or nil pointer."
|
|
Make clear there is no []byte reuse bug in SearchFilesByContent
Closes #1474
See merge request gitlab-org/gitaly!1055
|
|
|
|
This change will statically compile the jaeger client into gitaly by
default. This will allow support staff to enable the GITLAB_TRACING
variable to investigate issues further.
Note that by default, this will not enable tracing, only compile the
client in.
|
|
|
|
Sync master with the recent security release
See merge request gitlab-org/gitaly!1062
|
|
|
|
|
|
|
|
Switch from commitsSender to chunker
See merge request gitlab-org/gitaly!1060
|
|
|
|
Add glProjectName to logs
See merge request gitlab-org/gitaly!1049
|
|
|
|
|
|
Avoid unsafe use of scanner.Bytes() in ref name RPC's
Closes #1473
See merge request gitlab-org/gitaly!1054
|
|
|
|
|
|
|
|
|
|
|