Merge branch 'issue362' into 'master'

TCP health probes shouldn't be logged at INFO level

Closes #362

See merge request openconnect/ocserv!224
This commit is contained in:
Alan Jowett
2020-10-07 19:39:40 +00:00
2 changed files with 3 additions and 1 deletions

View File

@@ -192,6 +192,8 @@ const char *discon_reason_to_str(unsigned reason)
return "unspecified error";
case REASON_SESSION_TIMEOUT:
return "session timeout";
case REASON_HEALTH_PROBE:
return "TCP health probe";
default:
snprintf(tmp, sizeof(tmp), "unknown (%u)", reason);
return tmp;

View File

@@ -117,7 +117,7 @@ void remove_proc(main_server_st * s, struct proc_st *proc, unsigned flags)
}
}
mslog(s, proc, LOG_INFO, "user disconnected (reason: %s, rx: %"PRIu64", tx: %"PRIu64")",
mslog(s, proc, (proc->discon_reason != REASON_HEALTH_PROBE ? LOG_INFO : LOG_DEBUG), "user disconnected (reason: %s, rx: %"PRIu64", tx: %"PRIu64")",
discon_reason_to_str(proc->discon_reason), proc->bytes_in, proc->bytes_out);
pid = remove_from_script_list(s, proc);