[gnome-online-accounts] identity: don't ignore almost all renewal requests



commit 09e110375f3c1a6af7abf4f462041c9d61c813d2
Author: Ray Strode <rstrode redhat com>
Date:   Tue Sep 22 14:38:29 2015 -0400

    identity: don't ignore almost all renewal requests
    
    There's a bug in the code where we ignore all renewal requests for
    objects we already know about.
    
    This means we'll only ever honor renewal requests that happen almost
    immediately after start up.
    
    This commit fixes the bug, so the only renewal requests that aren't
    honored, are those that have been specifically disabled by the user.
    This was clearly the original intention of the buggy code.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=756495

 src/goaidentity/goaidentityservice.c |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)
---
diff --git a/src/goaidentity/goaidentityservice.c b/src/goaidentity/goaidentityservice.c
index 27b6cbe..f488767 100644
--- a/src/goaidentity/goaidentityservice.c
+++ b/src/goaidentity/goaidentityservice.c
@@ -751,16 +751,17 @@ on_identity_needs_renewal (GoaIdentityManager *identity_manager,
 
   principal = goa_identity_get_identifier (identity);
 
-  g_debug ("GoaIdentityService: identity %s needs renewal", principal);
-
   object = find_object_with_principal (self, principal, TRUE);
 
-  if (object != NULL)
+  if (object != NULL && should_ignore_object (self, object))
     {
-      should_ignore_object (self, object);
+      g_debug ("GoaIdentityService: ignoring identity %s that says it needs renewal", principal);
+
       return;
     }
 
+  g_debug ("GoaIdentityService: identity %s needs renewal", principal);
+
   goa_identity_manager_renew_identity (GOA_IDENTITY_MANAGER
                                        (self->priv->identity_manager),
                                        identity,


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