[SSSD] [PATCH] Clarify debug message about initgroups and subdomains

Sumit Bose sbose at redhat.com
Thu Nov 8 10:11:20 UTC 2012


Hi,

this patch fixes https://fedorahosted.org/sssd/ticket/1610 by changing
the debug message. See commit message for details.

bye,
Sumit
-------------- next part --------------
From 2d87f59dae27587028db1b6b9183fff052da1c81 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Thu, 8 Nov 2012 10:11:15 +0100
Subject: [PATCH] Clarify debug message about initgroups and subdomains

The initgroups request is not handled by the IPA provider for
subdomain users on purpose because the group membership information is
not available on the IPA server but will be directly written to the
cache when the PAC of the user is processed. The old generic debug
message "Invalid sub-domain request type" might be misleading.

This patch adds a specific message for the initgroups case "Initgroups
requests are not handled by the IPA provider but are resolved by the
responder directly from the cache." and increase the debug level so
that typically this message is not shown anymore because it is expected
behaviour.

Fixes https://fedorahosted.org/sssd/ticket/1610
---
 src/providers/ipa/ipa_subdomains_id.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/src/providers/ipa/ipa_subdomains_id.c b/src/providers/ipa/ipa_subdomains_id.c
index 5a4ab40..563fdf2 100644
--- a/src/providers/ipa/ipa_subdomains_id.c
+++ b/src/providers/ipa/ipa_subdomains_id.c
@@ -99,6 +99,13 @@ struct tevent_req *ipa_get_subdomain_account_info_send(TALLOC_CTX *memctx,
                                        SDAP_OPTS_GROUP, NULL,
                                        &state->attrs, NULL);
             break;
+        case BE_REQ_INITGROUPS:
+            ret = ENOTSUP;
+            DEBUG(SSSDBG_TRACE_FUNC, ("Initgroups requests are not handled " \
+                                      "by the IPA provider but are resolved " \
+                                      "by the responder directly from the " \
+                                      "cache.\n"));
+            break;
         default:
             ret = EINVAL;
             DEBUG(SSSDBG_OP_FAILURE, ("Invalid sub-domain request type.\n"));
-- 
1.7.7.6



More information about the sssd-devel mailing list