Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Commit

Permalink
injector: restrict envoy admin access to localhost
Browse files Browse the repository at this point in the history
The Envoy admin interface allows to perform destructive
operations as well as exposes sensitive information.
Access to the portal should be restricted to within
the localhost.
More info can be found at
https://www.envoyproxy.io/docs/envoy/latest/operations/admin

Port forwarding to the proxy pod to access the admin portal
over localhost will work as expected.

Signed-off-by: Shashank Ram <shashr2204@gmail.com>
  • Loading branch information
shashankram authored and nojnhuh committed Mar 12, 2021
1 parent 2d13cfe commit b956469
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pkg/injector/envoy_config.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ func getEnvoyConfigYAML(config envoyBootstrapConfigMeta, cfg configurator.Config
"access_log_path": "/dev/stdout",
"address": map[string]interface{}{
"socket_address": map[string]string{
"address": "0.0.0.0",
"address": constants.LocalhostIPAddress,
"port_value": strconv.Itoa(config.EnvoyAdminPort),
},
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ admin:
access_log_path: /dev/stdout
address:
socket_address:
address: 0.0.0.0
address: 127.0.0.1
port_value: "15000"
dynamic_resources:
ads_config:
Expand Down

0 comments on commit b956469

Please sign in to comment.