Skip to content

Commit c404166

Browse files
committed
✨feature: Forward all known GitHub headers stakater#105
1 parent 39cc2e9 commit c404166

File tree

3 files changed

+12
-6
lines changed

3 files changed

+12
-6
lines changed

.VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
version: v0.2.80
1+
version: v0.2.81

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
BUILDER ?= gitwebhookproxy-builder
66
BINARY ?= GitWebhookProxy
7-
DOCKER_IMAGE ?= stakater/gitwebhookproxy
7+
DOCKER_IMAGE ?= maxisam/gitwebhookproxy
88
# Default value "dev"
99
DOCKER_TAG ?= dev
1010
REPOSITORY = ${DOCKER_IMAGE}:${DOCKER_TAG}

pkg/parser/parser.go

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,17 @@ func Parse(req *http.Request, provider providers.Provider) (*providers.Hook, err
1414
}
1515

1616
for _, header := range provider.GetHeaderKeys() {
17-
if req.Header.Get(header) != "" {
18-
hook.Headers[header] = req.Header.Get(header)
19-
continue
17+
if req.Header.Get(header) == "" {
18+
return nil, errors.New("Required header '" + header + "' not found in Request")
2019
}
21-
return nil, errors.New("Required header '" + header + "' not found in Request")
20+
21+
// Store required headers in the expected casing
22+
hook.Headers[header] = req.Header.Get(header)
23+
}
24+
25+
for header := range req.Header {
26+
// Store the rest of the headers in any casing
27+
hook.Headers[header] = req.Header.Get(header)
2228
}
2329

2430
if body, err := ioutil.ReadAll(req.Body); err != nil {

0 commit comments

Comments
 (0)