LDEV-4406 - improve mail watcher itself

lucee · Dec 18, 2023 · 2c961ee · 2c961ee
1 parent f98aab0
commit 2c961ee
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/core/src/main/cfml/context/gateway/MailWatcher.cfc b/core/src/main/cfml/context/gateway/MailWatcher.cfc
@@ -29,15 +29,15 @@
 
 	public void function start() output=false localmode=true {
 
-		while ( getState() EQ "stopping" ) {
+		if ( getState() EQ "stopping" ) {
 			sleep(10);
 		}
 		variables.state="running";
 		writeLog ( text="start", file=variables.logFile, type="information" );
 		var last =now();
 		var mail = "";
 
-		while ( variables.state EQ "running" ){
+		while ( variables.state EQ "running" ) {
 			try {
 				mails = getMailsNewerThan( config.server, config.port, config.username, config.password, config.attachmentpath, last);
 				for ( el in mails ) {
@@ -52,8 +52,10 @@
 			if ( getState() != "running" ) {
 				break;
 			}
-			sleep( variables.config.interval );
+			sleep( variables.config.interval?:10 );
 		}
+		// set to stopped when we leave
+		variables.setState( "stopped" );
 	}
 
 	public array function getMailsNewerThan( required string server, required numeric port, required string user, required string pass,
@@ -86,6 +88,12 @@
 	public void function stop() output=false {
 		writeLog ( text="stop", file="MailWatcher", type="information" );
 		variables.state="stopping";
+
+		sleep( (variables.config.interval?:10)+10 );
+		// should be stopped, so we guess it is blockes somehow, because it will not run again, we can ignore it
+		if (getState() EQ "stopping" ) {
+			variables.state="stopping";
+		}
 	}
 
 	public void function restart() output=false {

diff --git a/loader/build.xml b/loader/build.xml
@@ -2,7 +2,7 @@
 <project default="core" basedir="." name="Lucee" 
   xmlns:resolver="antlib:org.apache.maven.resolver.ant">
 
-  <property name="version" value="6.0.1.43-SNAPSHOT"/>
+  <property name="version" value="6.0.1.44-SNAPSHOT"/>
 
   <taskdef uri="antlib:org.apache.maven.resolver.ant" resource="org/apache/maven/resolver/ant/antlib.xml">
     <classpath>

diff --git a/loader/pom.xml b/loader/pom.xml
@@ -3,7 +3,7 @@
 
   <groupId>org.lucee</groupId>
   <artifactId>lucee</artifactId>
-  <version>6.0.1.43-SNAPSHOT</version>
+  <version>6.0.1.44-SNAPSHOT</version>
   <packaging>jar</packaging>
 
   <name>Lucee Loader Build</name>