add asyncTimeout and fix rollover size not being applied correctly

changelog.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ----
 ## [Unreleased]
+### Added
+* Added `asyncTimeout` setting with default of 5000ms
+
+### Fixed
+* Fixed rollover size not being applied correctly on the ILM policy
 
 ## [3.2.4] - 11-21-2023
 ### Fixed

models/logging/LogstashAppender.cfc

@@ -72,7 +72,9 @@ component
 			"migrateIndices"        : false,
 			// Whether to throw an error if an attempt to save a log entry fails
 			"throwOnError"          : true,
-			"async"                 : false
+			"async"                 : false,
+			// Timeout, in ms, to allow async threads to exist - otherwise they default to 0
+			"asyncTimeout"          : 5000
 		};
 
 		for ( var configKey in structKeyArray( instance.Defaults ) ) {
@@ -150,7 +152,7 @@ component
 		try {
 			var document = newDocument().new( index = getProperty( "dataStream" ), properties = logObj );
 			if( getProperty( "async" ) ){
-				variables.asyncManager.newFuture().run( () => document.create() );
+				variables.asyncManager.newFuture().withTimeout( getProperty( "asyncTimeout" ) ).run( () => document.create() );
 			} else {
 				document.create();
 			}
@@ -364,7 +366,7 @@ component
 		if ( propertyExists( "lifecyclePolicy" ) ) {
 			policyBuilder.setPhases( getProperty( "lifecyclePolicy" ) );
 		} else {
-			policyBuilder.withDeletion( age = getProperty( "retentionDays" ) );
+			policyBuilder.hotPhase( rollover=getProperty( "rolloverSize" ) ).withDeletion( age = getProperty( "retentionDays" ) );
 		}
 
 		policyBuilder.save();