Commit fc35892a authored by Douwe Maan's avatar Douwe Maan

Merge branch 'always-allow-prometheus-access-in-dev' into 'master'

In dev, always allow access to health endpoints from localhost

See merge request gitlab-org/gitlab-ce!29930
parents 2b9ddc2f 26140f91
...@@ -14,6 +14,10 @@ module RequiresWhitelistedMonitoringClient ...@@ -14,6 +14,10 @@ module RequiresWhitelistedMonitoringClient
end end
def client_ip_whitelisted? 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) } ip_whitelist.any? { |e| e.include?(Gitlab::RequestContext.client_ip) }
end end
......
---
title: Always allow access to health endpoints from localhost in dev
merge_request: 29930
author:
type: other
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment