diff options
author | Andrew Newdigate <andrew@gitlab.com> | 2019-06-26 09:27:19 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2019-06-26 09:27:19 +0000 |
commit | 26140f91853f6e48ca6294441dc4bd92a43b4d86 (patch) | |
tree | 3e10e2b682a4356b27de1beeb0f2a37d424119fa | |
parent | 2b9ddc2f99bc0a49967c9ccc5b79ccc53e7559b4 (diff) | |
download | gitlab-ce-26140f91853f6e48ca6294441dc4bd92a43b4d86.tar.gz |
In dev, always allow access to health endpoints from localhost
This change will allow developers to easily hook up a Prometheus
instance to their local development instance, without making any
configuration changes.
-rw-r--r-- | app/controllers/concerns/requires_whitelisted_monitoring_client.rb | 4 | ||||
-rw-r--r-- | changelogs/unreleased/always-allow-prometheus-access-in-dev.yml | 5 |
2 files changed, 9 insertions, 0 deletions
diff --git a/app/controllers/concerns/requires_whitelisted_monitoring_client.rb b/app/controllers/concerns/requires_whitelisted_monitoring_client.rb index 426f224d26b..f47ead2f0da 100644 --- a/app/controllers/concerns/requires_whitelisted_monitoring_client.rb +++ b/app/controllers/concerns/requires_whitelisted_monitoring_client.rb @@ -14,6 +14,10 @@ module RequiresWhitelistedMonitoringClient end def client_ip_whitelisted? + # Always allow developers to access http://localhost:3000/-/metrics for + # debugging purposes + return true if Rails.env.development? && request.local? + ip_whitelist.any? { |e| e.include?(Gitlab::RequestContext.client_ip) } end diff --git a/changelogs/unreleased/always-allow-prometheus-access-in-dev.yml b/changelogs/unreleased/always-allow-prometheus-access-in-dev.yml new file mode 100644 index 00000000000..acd944ea684 --- /dev/null +++ b/changelogs/unreleased/always-allow-prometheus-access-in-dev.yml @@ -0,0 +1,5 @@ +--- +title: Always allow access to health endpoints from localhost in dev +merge_request: 29930 +author: +type: other |