Skip to content
Snippets Groups Projects
Commit 7f11029b authored by Dominik Hebeler's avatar Dominik Hebeler
Browse files

disallow public access to metrics endpoint

parent fb5f71ff
No related branches found
No related tags found
3 merge requests!1895Development,!1876Development,!1875Resolve "Update"
......@@ -3,6 +3,7 @@
use Illuminate\Support\Facades\Redis;
use Jenssegers\Agent\Agent;
use Prometheus\RenderTextFormat;
use Illuminate\Http\Request;
/*
|--------------------------------------------------------------------------
......@@ -320,7 +321,27 @@ Route::group(
});
});
Route::get('metrics', function () {
Route::get('metrics', function (Request $request) {
// Only allow access to metrics from within our network
$ip = $request->ip();
$allowedNetworks = [
"10.",
"172.",
"192.",
"127.0.0.1",
];
$allowed = false;
foreach($allowedNetworks as $part){
if(stripos($ip, $part) === 0){
$allowed = true;
}
}
if(!$allowed){
abort(401);
}
$registry = \Prometheus\CollectorRegistry::getDefault();
$renderer = new RenderTextFormat();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment