Skip to content

Commit a7f3f20

Browse files
allan2CBenoit
andauthored
fix(dgw): split TlsVerifyStrict warning for absent and false (#1520)
Co-authored-by: Benoît Cortier <3809077+CBenoit@users.noreply.github.com>
1 parent 4cd35eb commit a7f3f20

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

devolutions-gateway/src/service.rs

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,16 @@ impl GatewayService {
6969
);
7070
}
7171

72-
if matches!(conf_file.tls_verify_strict, None | Some(false)) {
73-
warn!("TlsVerifyStrict option is absent or set to false. This may hide latent issues.");
74-
let _ = SYSTEM_LOGGER.emit(sysevent_codes::tls_verify_strict_disabled(
75-
"TlsVerifyStrict option is absent or set to false",
76-
));
72+
match conf_file.tls_verify_strict {
73+
None => {
74+
warn!("TlsVerifyStrict option is not set, defaulting to false. This may hide latent issues.");
75+
let _ = SYSTEM_LOGGER.emit(sysevent_codes::tls_verify_strict_disabled("compat"));
76+
}
77+
Some(false) => {
78+
warn!("TlsVerifyStrict option is explicitly set to false. This may hide latent issues.");
79+
let _ = SYSTEM_LOGGER.emit(sysevent_codes::tls_verify_strict_disabled("explicit"));
80+
}
81+
Some(true) => {}
7782
}
7883

7984
if let Some((cert_subject_name, hostname)) = conf_file

0 commit comments

Comments
 (0)