Skip to content

Commit 4b00981

Browse files
authored
Merge branch 'master' into feat/handle-event
2 parents fb2a165 + c404166 commit 4b00981

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

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)