Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-workhorse
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-workhorse
Commits
fe74d237
Commit
fe74d237
authored
Jun 05, 2019
by
Yoginth
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Fix Typos
parent
b572ab40
Changes
5
Hide whitespace changes
Inline
Side-by-side
Showing
5 changed files
with
6 additions
and
6 deletions
+6
-6
Makefile
Makefile
+1
-1
internal/api/api.go
internal/api/api.go
+2
-2
internal/objectstore/object.go
internal/objectstore/object.go
+1
-1
internal/objectstore/test/consts.go
internal/objectstore/test/consts.go
+1
-1
internal/sendurl/sendurl.go
internal/sendurl/sendurl.go
+1
-1
No files found.
Makefile
View file @
fe74d237
...
@@ -135,7 +135,7 @@ check-formatting: $(TARGET_SETUP) install-goimports
...
@@ -135,7 +135,7 @@ check-formatting: $(TARGET_SETUP) install-goimports
@
_support/validate-formatting.sh
$(LOCAL_GO_FILES)
@
_support/validate-formatting.sh
$(LOCAL_GO_FILES)
# Megacheck will tailor some responses given a minimum Go version, so pass that through the CLI
# Megacheck will tailor some responses given a minimum Go version, so pass that through the CLI
# Additionally, megacheck will not return failure exit codes unless explicit
e
ly told to via the
# Additionally, megacheck will not return failure exit codes unless explicitly told to via the
# `-simple.exit-non-zero` `-unused.exit-non-zero` and `-staticcheck.exit-non-zero` flags
# `-simple.exit-non-zero` `-unused.exit-non-zero` and `-staticcheck.exit-non-zero` flags
.PHONY
:
staticcheck
.PHONY
:
staticcheck
staticcheck
:
$(TARGET_SETUP) govendor-sync
staticcheck
:
$(TARGET_SETUP) govendor-sync
...
...
internal/api/api.go
View file @
fe74d237
...
@@ -187,7 +187,7 @@ func (api *API) newRequest(r *http.Request, suffix string) (*http.Request, error
...
@@ -187,7 +187,7 @@ func (api *API) newRequest(r *http.Request, suffix string) (*http.Request, error
authReq
.
Header
.
Del
(
"Trailers"
)
authReq
.
Header
.
Del
(
"Trailers"
)
authReq
.
Header
.
Del
(
"Upgrade"
)
authReq
.
Header
.
Del
(
"Upgrade"
)
// Also forward the Host header, which is excluded from the Header map by the http libary.
// Also forward the Host header, which is excluded from the Header map by the http lib
r
ary.
// This allows the Host header received by the backend to be consistent with other
// This allows the Host header received by the backend to be consistent with other
// requests not going through gitlab-workhorse.
// requests not going through gitlab-workhorse.
authReq
.
Host
=
r
.
Host
authReq
.
Host
=
r
.
Host
...
@@ -296,7 +296,7 @@ func passResponseBack(httpResponse *http.Response, w http.ResponseWriter, r *htt
...
@@ -296,7 +296,7 @@ func passResponseBack(httpResponse *http.Response, w http.ResponseWriter, r *htt
bytesTotal
.
Add
(
float64
(
responseBody
.
Len
()))
bytesTotal
.
Add
(
float64
(
responseBody
.
Len
()))
for
k
,
v
:=
range
httpResponse
.
Header
{
for
k
,
v
:=
range
httpResponse
.
Header
{
// Accomodate broken clients that do case-sensitive header lookup
// Accom
m
odate broken clients that do case-sensitive header lookup
if
k
==
"Www-Authenticate"
{
if
k
==
"Www-Authenticate"
{
w
.
Header
()[
"WWW-Authenticate"
]
=
v
w
.
Header
()[
"WWW-Authenticate"
]
=
v
}
else
{
}
else
{
...
...
internal/objectstore/object.go
View file @
fe74d237
...
@@ -17,7 +17,7 @@ import (
...
@@ -17,7 +17,7 @@ import (
// httpTransport defines a http.Transport with values
// httpTransport defines a http.Transport with values
// that are more restrictive than for http.DefaultTransport,
// that are more restrictive than for http.DefaultTransport,
// they define shorter TLS Handshake, and more agressive connection closing
// they define shorter TLS Handshake, and more ag
g
ressive connection closing
// to prevent the connection hanging and reduce FD usage
// to prevent the connection hanging and reduce FD usage
var
httpTransport
=
tracing
.
NewRoundTripper
(
correlation
.
NewInstrumentedRoundTripper
(
&
http
.
Transport
{
var
httpTransport
=
tracing
.
NewRoundTripper
(
correlation
.
NewInstrumentedRoundTripper
(
&
http
.
Transport
{
Proxy
:
http
.
ProxyFromEnvironment
,
Proxy
:
http
.
ProxyFromEnvironment
,
...
...
internal/objectstore/test/consts.go
View file @
fe74d237
package
test
package
test
// Some useful
l
const for testing purpose
// Some useful const for testing purpose
const
(
const
(
// ObjectContent an example textual content
// ObjectContent an example textual content
ObjectContent
=
"TEST OBJECT CONTENT"
ObjectContent
=
"TEST OBJECT CONTENT"
...
...
internal/sendurl/sendurl.go
View file @
fe74d237
...
@@ -37,7 +37,7 @@ var rangeHeaderKeys = []string{
...
@@ -37,7 +37,7 @@ var rangeHeaderKeys = []string{
// httpTransport defines a http.Transport with values
// httpTransport defines a http.Transport with values
// that are more restrictive than for http.DefaultTransport,
// that are more restrictive than for http.DefaultTransport,
// they define shorter TLS Handshake, and more agressive connection closing
// they define shorter TLS Handshake, and more ag
g
ressive connection closing
// to prevent the connection hanging and reduce FD usage
// to prevent the connection hanging and reduce FD usage
var
httpTransport
=
tracing
.
NewRoundTripper
(
correlation
.
NewInstrumentedRoundTripper
(
&
http
.
Transport
{
var
httpTransport
=
tracing
.
NewRoundTripper
(
correlation
.
NewInstrumentedRoundTripper
(
&
http
.
Transport
{
Proxy
:
http
.
ProxyFromEnvironment
,
Proxy
:
http
.
ProxyFromEnvironment
,
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment