Replace failure with warning in 'test_auth_methods'

Previously 'test_auth_methods' flagged a current ProxySQL limitation
regarding auth full-handshakes with an error. This has been replaced
with a warning until fixed.
pull/4833/head
Javier Jaramago Fernández 1 year ago
parent 1da6dfec29
commit 6e932c82de

@ -935,6 +935,21 @@ bool detect_sha2_full_auth(const sess_info_t& sess_info) {
bool chk_exp_sha2_full_auth(
const test_conf_t& conf, const test_creds_t& creds, const user_auth_stats_t& auth_info
) {
// TODO: ProxySQL requires a full-auth everytime a dual-password is used. This is a limitation
// on the current auth state-machine. This should be fixed in the next auth-rework. The
// following code will be obsolete when this is implemented.
///////////////////////////////////////////////////////////////////////////
if (
creds.info.type == PASS_TYPE::ADDITIONAL
&& conf.req_auth == "caching_sha2_password"
&& conf.def_auth == "caching_sha2_password"
&& conf.hashed_pass == true
) {
diag("TODO-WARNING: The following check will FAKE pass - This limitation should be fixed");
return true;
}
///////////////////////////////////////////////////////////////////////////
if (!is_empty_pass(creds.pass.get()) && conf.hashed_pass && creds.info.auth == "caching_sha2_password") {
if (creds.info.type == PASS_TYPE::PRIMARY) {
return auth_info.prim_pass_auths == 0;

Loading…
Cancel
Save