URL: https://github.com/SSSD/sssd/pull/5600 Author: DeepakDas7 Title: #5600: SSSD Log: log_bad_address_msg_modification Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5600/head:pr5600 git checkout pr5600
From 950ba7ce6f715282b15880212b95e6938e9d518e Mon Sep 17 00:00:00 2001 From: Deepak Das <[email protected]> Date: Fri, 23 Apr 2021 11:11:02 +0530 Subject: [PATCH] SSSD Log: log_bad_address_msg_modification Improve Log Containing Bad Address string Resolves: https://github.com/SSSD/sssd/issues/5577 --- src/providers/ad/ad_gpo.c | 6 ++++-- src/providers/data_provider_fo.c | 10 ++++++++-- src/providers/ldap/sdap_async_connection.c | 11 +++++++++-- 3 files changed, 21 insertions(+), 6 deletions(-) diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c index 4ef6a7219c..28c58307bf 100644 --- a/src/providers/ad/ad_gpo.c +++ b/src/providers/ad/ad_gpo.c @@ -2448,9 +2448,11 @@ ad_gpo_process_gpo_done(struct tevent_req *subreq) if (ret != EOK && ret != ENOENT) { DEBUG(SSSDBG_OP_FAILURE, - "Unable to get GPO list: [%d](%s)\n", - ret, sss_strerror(ret)); + "Unable to get GPO list from server %s: [%d](%s)\n", + state->ad_hostname, ret, sss_strerror(ret)); + goto done; + } else if (ret == ENOENT) { DEBUG(SSSDBG_TRACE_FUNC, "No GPOs found that apply to this system.\n"); diff --git a/src/providers/data_provider_fo.c b/src/providers/data_provider_fo.c index 0dfbb04b0f..1c47a1dd28 100644 --- a/src/providers/data_provider_fo.c +++ b/src/providers/data_provider_fo.c @@ -575,8 +575,14 @@ static void be_resolve_server_done(struct tevent_req *subreq) return; fail: - DEBUG(SSSDBG_TRACE_LIBS, - "Server resolution failed: [%d]: %s\n", ret, sss_strerror(ret)); + if (ret == EFAULT || ret == EIO || ret == EPERM) { + DEBUG(SSSDBG_TRACE_LIBS, + "Server [%s] resolution failed: [%d]: %s\n", + fo_get_server_name(state->srv), ret, sss_strerror(ret)); + } else { + DEBUG(SSSDBG_TRACE_LIBS, + "Server resolution failed: [%d]: %s\n", ret, sss_strerror(ret)); + } state->svc->first_resolved = NULL; tevent_req_error(req, ret); } diff --git a/src/providers/ldap/sdap_async_connection.c b/src/providers/ldap/sdap_async_connection.c index eead3f119a..8bd80d4ccf 100644 --- a/src/providers/ldap/sdap_async_connection.c +++ b/src/providers/ldap/sdap_async_connection.c @@ -1274,8 +1274,15 @@ static void sdap_kinit_done(struct tevent_req *subreq) } - DEBUG(SSSDBG_CONF_SETTINGS, - "Could not get TGT: %d [%s]\n", result, sss_strerror(result)); + if (result == EFAULT || result == EIO || result == EPERM) { + DEBUG(SSSDBG_CONF_SETTINGS, + "Could not get TGT from server %s: %d [%s]\n", + fo_get_server_name(state->kdc_srv), result, sss_strerror(result)); + } else { + DEBUG(SSSDBG_CONF_SETTINGS, + "Could not get TGT: %d [%s]\n", result, sss_strerror(result)); + } + tevent_req_error(req, ERR_AUTH_FAILED); }
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/[email protected] Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
