[java-opensaml] branch master updated: OSJ-216: Manual metadata resolver refresh() is starting multiple refresh cycles

Brent Putman putmanb at georgetown.edu
Fri Sep 8 18:29:22 EDT 2017


This is an automated email from the git hooks/post-receive script.

putmanb pushed a commit to branch master
in repository java-opensaml.

View the commit online:
http://git.shibboleth.net/view/?p=java-opensaml.git;a=commit;h=bf7849b7f51993e7ae3668dc77a2f2b63aa6b275

The following commit(s) were added to refs/heads/master by this push:
       new  bf7849b   OSJ-216: Manual metadata resolver refresh() is starting multiple refresh cycles
bf7849b is described below

commit bf7849b7f51993e7ae3668dc77a2f2b63aa6b275
Author: Brent Putman <putmanb at georgetown.edu>
AuthorDate: Fri Sep 8 18:29:13 2017 -0400

    OSJ-216: Manual metadata resolver refresh() is starting multiple refresh
    cycles
---
 .../metadata/resolver/impl/AbstractReloadingMetadataResolver.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/opensaml-saml-impl/src/main/java/org/opensaml/saml/metadata/resolver/impl/AbstractReloadingMetadataResolver.java b/opensaml-saml-impl/src/main/java/org/opensaml/saml/metadata/resolver/impl/AbstractReloadingMetadataResolver.java
index c2adf88..f935691 100644
--- a/opensaml-saml-impl/src/main/java/org/opensaml/saml/metadata/resolver/impl/AbstractReloadingMetadataResolver.java
+++ b/opensaml-saml-impl/src/main/java/org/opensaml/saml/metadata/resolver/impl/AbstractReloadingMetadataResolver.java
@@ -333,6 +333,12 @@ public abstract class AbstractReloadingMetadataResolver extends AbstractBatchMet
         trackRefreshSuccess = false;
 
         try {
+            // A manual refresh() must cancel the previously-scheduled future task, since will (re)schedule its own.
+            // If this execution *is* the task, it's ok to cancel ourself, we're already running.
+            if (refreshMetadataTask != null) {
+                refreshMetadataTask.cancel();
+            }
+            
             now = new DateTime(ISOChronology.getInstanceUTC());
             mdId = getMetadataIdentifier();
 

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list