[gnome-online-accounts/wip/rishi/kcm-supports-multiple-identities] kerberos-identity-manager: Clarify an ambiguous debug log about KCM




commit 89ac2054cb8d82294a2ad664ad298abd32bad8be
Author: Debarshi Ray <debarshir gnome org>
Date:   Wed Oct 12 22:59:34 2022 +0200

    kerberos-identity-manager: Clarify an ambiguous debug log about KCM
    
    Kerberos KCM credential caches do support multiple identities and a lot
    of users use KCM these days because it's the default on Fedora.  Hence,
    it's better not to have a debug log that implies that the code wasn't
    expecting KCM and is making assumptions about it - KCM is definitely a
    supported Kerberos cache type.
    
    https://gitlab.gnome.org/GNOME/gnome-online-accounts/-/merge_requests/104

 src/goaidentity/goakerberosidentitymanager.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
---
diff --git a/src/goaidentity/goakerberosidentitymanager.c b/src/goaidentity/goakerberosidentitymanager.c
index caed5ae2..4fcb132c 100644
--- a/src/goaidentity/goakerberosidentitymanager.c
+++ b/src/goaidentity/goakerberosidentitymanager.c
@@ -834,7 +834,9 @@ get_new_credentials_cache (GoaKerberosIdentityManager *self,
                self->credentials_cache_type);
       supports_multiple_identities = FALSE;
     }
-  else if (g_strcmp0 (self->credentials_cache_type, "DIR") == 0 || g_strcmp0 (self->credentials_cache_type, 
"KEYRING") == 0)
+  else if (g_strcmp0 (self->credentials_cache_type, "DIR") == 0
+           || g_strcmp0 (self->credentials_cache_type, "KCM") == 0
+           || g_strcmp0 (self->credentials_cache_type, "KEYRING") == 0)
     {
       g_debug ("GoaKerberosIdentityManager: credential cache type %s supports cache collections", 
self->credentials_cache_type);
       supports_multiple_identities = TRUE;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]