Skip to content

Commit

Permalink
Remove repetitive log messages (#154)
Browse files Browse the repository at this point in the history
If AXParameter or environment variable setup fails,
a detailed message is already logged in the failing function.
There is no need to repeat that message with less detail at the call site.

Co-authored-by: Mattias Axelsson <mattiaax@axis.com>
  • Loading branch information
github-actions[bot] and killenheladagen authored Apr 30, 2024
1 parent af679d6 commit eaf94b9
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions app/dockerdwrapperwithcompose.c
Original file line number Diff line number Diff line change
Expand Up @@ -796,17 +796,13 @@ int main(int argc, char** argv) {
allow_dockerd_to_start(&app_state, true);

app_state.param_handle = setup_axparameter(&app_state);
if (!app_state.param_handle) {
log_error("Error in setup_axparameter");
if (!app_state.param_handle)
return EX_SOFTWARE;
}

log_debug_set(is_app_log_level_debug(app_state.param_handle));

if (!set_env_variables()) {
log_error("Failed to set environment variables");
if (!set_env_variables())
return EX_SOFTWARE;
}

init_signals();

Expand Down

0 comments on commit eaf94b9

Please sign in to comment.