From 5f9f8c0e529bceee589277c5475508e91f146b64 Mon Sep 17 00:00:00 2001 From: "Jeroen van Meeuwen (Kolab Systems)" Date: Sun, 16 Sep 2012 19:23:51 +0100 Subject: Re-enable and tweak the kolab_smtp_access_policy cache --- bin/kolab_smtp_access_policy.py | 39 +++++++++++---------------------------- 1 file changed, 11 insertions(+), 28 deletions(-) diff --git a/bin/kolab_smtp_access_policy.py b/bin/kolab_smtp_access_policy.py index bd94925..bce2614 100755 --- a/bin/kolab_smtp_access_policy.py +++ b/bin/kolab_smtp_access_policy.py @@ -629,24 +629,15 @@ class PolicyRequest(object): # Got a final answer here, do the caching thing. if not cache == False: - records = cache_select( + record_id = cache_update( function='verify_sender', sender=self.sender, recipients=self.recipients, + result=(int)(False), sasl_username=self.sasl_username, sasl_sender=self.sasl_sender ) - if not len(records) == len(self.recipients): - record_id = cache_insert( - function='verify_sender', - sender=self.sender, - recipients=self.recipients, - result=(int)(False), - sasl_username=self.sasl_username, - sasl_sender=self.sasl_sender - ) - sender_is_delegate = False else: @@ -1051,24 +1042,15 @@ class PolicyRequest(object): sender_verified = True if not cache == False: - records = cache_select( + record_id = cache_update( function='verify_sender', sender=self.sender, recipients=self.recipients, + result=(int)(sender_verified), sasl_username=self.sasl_username, - sasl_sender=self.sasl_sender, + sasl_sender=self.sasl_sender ) - if len(records) == len(self.recipients): - record_id = cache_insert( - function='verify_sender', - sender=self.sender, - recipients=self.recipients, - result=(int)(sender_verified), - sasl_username=self.sasl_username, - sasl_sender=self.sasl_sender - ) - return sender_verified def cache_cleanup(): @@ -1085,11 +1067,9 @@ def cache_cleanup(): def cache_init(): global cache, cache_expire, session - return False - if conf.has_section('kolab_smtp_access_policy'): - if conf.has_option('kolab_smtp_access_policy', 'uri'): - cache_uri = conf.get('kolab_smtp_access_policy', 'uri') + if conf.has_option('kolab_smtp_access_policy', 'cache_uri'): + cache_uri = conf.get('kolab_smtp_access_policy', 'cache_uri') cache = True if conf.has_option('kolab_smtp_access_policy', 'retention'): cache_expire = (int)( @@ -1098,7 +1078,10 @@ def cache_init(): 'retention' ) ) - + elif conf.has_option('kolab_smtp_access_policy', 'uri'): + log.warning(_("The 'uri' setting in the kolab_smtp_access_policy section is soon going to be deprecated in favor of 'cache_uri'")) + cache_uri = conf.get('kolab_smtp_access_policy', 'uri') + cache = True else: return False else: -- cgit v0.12