Skip to content

Commit

Permalink
fix(webserver): correct enabled check
Browse files Browse the repository at this point in the history
It was checking twice for prometheus configuration rather than metrics and then prometheus.

Co-authored-by: Federico Di Pierro <nierro92@gmail.com>
Signed-off-by: Samuel Gaist <samuel.gaist@idiap.ch>
  • Loading branch information
sgaist and FedeDP authored Mar 19, 2024
1 parent 47ab618 commit 74dd00f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion userspace/falco/app/actions/start_webserver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ falco::app::run_result falco::app::actions::start_webserver(falco::app::state& s
+ ssl_option + "\n");

std::vector<libs::metrics::libs_metrics_collector> metrics_collectors;
if (s.config->m_metrics_prometheus_enabled && s.config->m_metrics_prometheus_enabled)
if (s.config->m_metrics_enabled && s.config->m_metrics_prometheus_enabled)
{
for (const auto& source_info: s.source_infos)
{
Expand Down

0 comments on commit 74dd00f

Please sign in to comment.