Skip to content

Commit

Permalink
envoy: Add response headers access logging
Browse files Browse the repository at this point in the history
Use cilium-envoy image that adds response headers to response access
log messages.

Signed-off-by: Jarno Rajahalme <jarno@isovalent.com>
  • Loading branch information
jrajahalme committed May 12, 2021
1 parent a9f18f3 commit 094d141
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion images/cilium/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ ARG CILIUM_RUNTIME_IMAGE=quay.io/cilium/cilium-runtime:8fe001a11f25ad9e6676c19b0

# cilium-envoy from github.com/cilium/proxy
#
FROM quay.io/cilium/cilium-envoy:b3bb275dcaf4a74ae956e52fa408b196c0d52152@sha256:f3c7645c0ff1d7552b79927023cf540f82b40263780644f75291ed0bb1965bee as cilium-envoy
FROM quay.io/cilium/cilium-envoy:1243d3855411ea5cfc3e8cfae0d552394754c186@sha256:3764904459830c2b8380922571a9f6d982795dc80cd9950ae600b8009604bbf0 as cilium-envoy

#
# Hubble CLI
Expand Down
4 changes: 3 additions & 1 deletion pkg/proxy/accesslog/record.go
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,9 @@ type LogRecordHTTP struct {
// Protocol is the HTTP protocol in use
Protocol string

// Headers are all HTTP headers present in the request
// Headers are all HTTP headers present in the request and response. Request records
// contain request headers, while response headers contain both request and response
// headers.
Headers http.Header

// MissingHeaders are HTTP request headers that were deemed missing from the request
Expand Down

0 comments on commit 094d141

Please sign in to comment.