Skip to content

Commit

Permalink
Merge pull request #34 from foomo/fix/collect-response-header
Browse files Browse the repository at this point in the history
fix: copy headers to response
  • Loading branch information
franklinkim authored Nov 26, 2024
2 parents ad3a673 + 67a1267 commit bc2e2e0
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions pkg/collect/collect.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,11 @@ func (c *Collect) gtagHandler(l *zap.Logger, w http.ResponseWriter, r *http.Requ
defer resp.Body.Close()

// copy headers
r.Header = resp.Header.Clone()
for key, values := range resp.Header {
for _, value := range values {
w.Header().Add(key, value)
}
}

if _, err := io.Copy(w, resp.Body); err != nil {
return err
Expand Down Expand Up @@ -174,7 +178,11 @@ func (c *Collect) mpv2Handler(l *zap.Logger, w http.ResponseWriter, r *http.Requ
defer resp.Body.Close()

// copy headers
r.Header = resp.Header.Clone()
for key, values := range resp.Header {
for _, value := range values {
w.Header().Add(key, value)
}
}

if _, err := io.Copy(w, resp.Body); err != nil {
return err
Expand Down

0 comments on commit bc2e2e0

Please sign in to comment.