some updates based on pr feedback

Author: heskew

integrationTests/utils/securityServices.ts

@@ -66,14 +66,15 @@ export async function startOcspResponder(port: number, certsPath: string): Promi
 			});
 		}, 2000);
 
-		proc.stdout?.on('data', (data) => {
+		// stdio is piped, so stdout/stderr are always defined
+		proc.stdout.on('data', (data) => {
 			// Log OCSP responder output for debugging
 			if (process.env.DEBUG_OCSP) {
 				console.log('OCSP responder:', data.toString());
 			}
 		});
 
-		proc.stderr?.on('data', (data) => {
+		proc.stderr.on('data', (data) => {
 			// Log errors for debugging
 			if (process.env.DEBUG_OCSP) {
 				console.error('OCSP responder error:', data.toString());
@@ -87,7 +88,10 @@ export async function startOcspResponder(port: number, certsPath: string): Promi
 			}
 		});
 
-		proc.on('exit', (code) => {
+		// Use 'close' instead of 'exit' to ensure all stdio streams are fully flushed
+		// and closed before handling the error - this prevents race conditions where we
+		// might reject before all error output has been captured
+		proc.on('close', (code) => {
 			if (timeout) {
 				clearTimeout(timeout);
 				reject(new Error(`OCSP responder exited with code ${code} before starting`));
@@ -109,7 +113,7 @@ export async function stopOcspResponder(ctx: OcspResponderContext): Promise<void
 			resolve();
 		}, 5000);
 
-		ctx.process.on('exit', () => {
+		ctx.process.on('close', () => {
 			clearTimeout(timeout);
 			resolve();
 		});