From 6cf80ca65c5511d5212b25a1d6cf4a60faa39764 Mon Sep 17 00:00:00 2001 From: Jordan Brown Date: Fri, 18 Dec 2020 11:20:45 -0500 Subject: [PATCH] Check validation status instead of assuming not --- .../Components/Deployment/DreamMaker.cs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Tgstation.Server.Host/Components/Deployment/DreamMaker.cs b/src/Tgstation.Server.Host/Components/Deployment/DreamMaker.cs index c031f55a4d..4d9f1aa9b4 100644 --- a/src/Tgstation.Server.Host/Components/Deployment/DreamMaker.cs +++ b/src/Tgstation.Server.Host/Components/Deployment/DreamMaker.cs @@ -238,13 +238,13 @@ namespace Tgstation.Server.Host.Components.Deployment await controller.Lifetime.WithToken(cancellationToken).ConfigureAwait(false); if (!controller.Lifetime.IsCompleted) - { - if (requireValidate) - throw new JobException(ErrorCode.DreamMakerNeverValidated); await controller.DisposeAsync().ConfigureAwait(false); - } validationStatus = controller.ApiValidationStatus; + + if (requireValidate && validationStatus == ApiValidationStatus.NeverValidated) + throw new JobException(ErrorCode.DreamMakerNeverValidated); + logger.LogTrace("API validation status: {0}", validationStatus); job.DMApiVersion = controller.DMApiVersion;