Move `if CONFIG.statistics_enabled` into the handler for the /metrics route

This commit is contained in:
Mateusz Bączek 2024-01-23 21:24:08 +01:00
parent 8d4c16c79c
commit 70754659e5
2 changed files with 6 additions and 7 deletions

View File

@ -29,6 +29,11 @@ module Invidious::Routes::API::V1::Misc
end end
def self.metrics(env) def self.metrics(env)
if !CONFIG.statistics_enabled
env.response.status_code = 204
return
end
env.response.content_type = "text/plain" env.response.content_type = "text/plain"
return String.build do |str| return String.build do |str|

View File

@ -310,13 +310,7 @@ module Invidious::Routing
# Misc # Misc
get "/api/v1/stats", {{namespace}}::Misc, :stats get "/api/v1/stats", {{namespace}}::Misc, :stats
if CONFIG.statistics_enabled get "/api/v1/metrics", {{namespace}}::Misc, :metrics
get "/api/v1/metrics", {{namespace}}::Misc, :metrics
else
get "/api/v1/metrics" do |env|
env.response.status_code = 204
end
end
get "/api/v1/playlists/:plid", {{namespace}}::Misc, :get_playlist get "/api/v1/playlists/:plid", {{namespace}}::Misc, :get_playlist
get "/api/v1/auth/playlists/:plid", {{namespace}}::Misc, :get_playlist get "/api/v1/auth/playlists/:plid", {{namespace}}::Misc, :get_playlist
get "/api/v1/mixes/:rdid", {{namespace}}::Misc, :mixes get "/api/v1/mixes/:rdid", {{namespace}}::Misc, :mixes