Fix GraphQL experimental feature being required

.github/workflows/ci-pipeline.yml

@@ -41,7 +41,6 @@ env:
   TGS_WEBPANEL_NODE_VERSION: 20.x
   TGS_TEST_GITHUB_TOKEN: ${{ secrets.LIVE_TESTS_TOKEN }}
   PACKAGING_PRIVATE_KEY_PASSPHRASE: ${{ secrets.PACKAGING_PRIVATE_KEY_PASSPHRASE }}
-  Internal__EnableGraphQL: true
 
 concurrency:
   group: "ci-${{ (github.event_name != 'push' && github.event_name != 'schedule' && github.event.inputs.pull_request_number) || github.run_id }}-${{ github.event_name }}"

build/Version.props

@@ -3,7 +3,7 @@
   <!-- Integration tests will ensure they match across the board -->
   <Import Project="WebpanelVersion.props" />
   <PropertyGroup>
-    <TgsCoreVersion>6.11.0</TgsCoreVersion>
+    <TgsCoreVersion>6.11.1</TgsCoreVersion>
     <TgsConfigVersion>5.3.0</TgsConfigVersion>
     <TgsRestVersion>10.10.0</TgsRestVersion>
     <TgsGraphQLVersion>0.2.0</TgsGraphQLVersion>

src/Tgstation.Server.Host/Core/Application.cs

@@ -295,55 +295,54 @@ void ConfigureNewtonsoftJsonSerializerSettingsForApi(JsonSerializerSettings sett
 			services.AddSingleton<IAbstractHttpClientFactory, AbstractHttpClientFactory>();
 
 			// configure graphql
-			if (postSetupServices.InternalConfiguration.EnableGraphQL)
-				services
-					.AddScoped<GraphQL.Subscriptions.ITopicEventReceiver, ShutdownAwareTopicEventReceiver>()
-					.AddGraphQLServer()
-					.AddAuthorization()
-					.ModifyOptions(options =>
-					{
-						options.EnsureAllNodesCanBeResolved = true;
-						options.EnableFlagEnums = true;
-					})
+			services
+				.AddScoped<GraphQL.Subscriptions.ITopicEventReceiver, ShutdownAwareTopicEventReceiver>()
+				.AddGraphQLServer()
+				.AddAuthorization()
+				.ModifyOptions(options =>
+				{
+					options.EnsureAllNodesCanBeResolved = true;
+					options.EnableFlagEnums = true;
+				})
 #if DEBUG
-					.ModifyCostOptions(options =>
-					{
-						options.EnforceCostLimits = false;
-					})
+				.ModifyCostOptions(options =>
+				{
+					options.EnforceCostLimits = false;
+				})
 #endif
-					.AddMutationConventions()
-					.AddInMemorySubscriptions(
-						new SubscriptionOptions
-						{
-							TopicBufferCapacity = 1024, // mainly so high for tests, not possible to DoS the server without authentication and some other access to generate messages
-						})
-					.AddGlobalObjectIdentification()
-					.AddQueryFieldToMutationPayloads()
-					.ModifyOptions(options =>
-					{
-						options.EnableDefer = true;
-					})
-					.ModifyPagingOptions(pagingOptions =>
+				.AddMutationConventions()
+				.AddInMemorySubscriptions(
+					new SubscriptionOptions
 					{
-						pagingOptions.IncludeTotalCount = true;
-						pagingOptions.RequirePagingBoundaries = false;
-						pagingOptions.DefaultPageSize = ApiController.DefaultPageSize;
-						pagingOptions.MaxPageSize = ApiController.MaximumPageSize;
+						TopicBufferCapacity = 1024, // mainly so high for tests, not possible to DoS the server without authentication and some other access to generate messages
 					})
-					.AddFiltering()
-					.AddSorting()
-					.AddHostTypes()
-					.AddErrorFilter<ErrorMessageFilter>()
-					.AddType<StandaloneNode>()
-					.AddType<LocalGateway>()
-					.AddType<RemoteGateway>()
-					.AddType<GraphQL.Types.UserName>()
-					.AddType<UnsignedIntType>()
-					.BindRuntimeType<Version, SemverType>()
-					.TryAddTypeInterceptor<RightsTypeInterceptor>()
-					.AddQueryType<Query>()
-					.AddMutationType<Mutation>()
-					.AddSubscriptionType<Subscription>();
+				.AddGlobalObjectIdentification()
+				.AddQueryFieldToMutationPayloads()
+				.ModifyOptions(options =>
+				{
+					options.EnableDefer = true;
+				})
+				.ModifyPagingOptions(pagingOptions =>
+				{
+					pagingOptions.IncludeTotalCount = true;
+					pagingOptions.RequirePagingBoundaries = false;
+					pagingOptions.DefaultPageSize = ApiController.DefaultPageSize;
+					pagingOptions.MaxPageSize = ApiController.MaximumPageSize;
+				})
+				.AddFiltering()
+				.AddSorting()
+				.AddHostTypes()
+				.AddErrorFilter<ErrorMessageFilter>()
+				.AddType<StandaloneNode>()
+				.AddType<LocalGateway>()
+				.AddType<RemoteGateway>()
+				.AddType<GraphQL.Types.UserName>()
+				.AddType<UnsignedIntType>()
+				.BindRuntimeType<Version, SemverType>()
+				.TryAddTypeInterceptor<RightsTypeInterceptor>()
+				.AddQueryType<Query>()
+				.AddMutationType<Mutation>()
+				.AddSubscriptionType<Subscription>();
 
 			void AddTypedContext<TContext>()
 				where TContext : DatabaseContext

tests/Tgstation.Server.Tests/Live/LiveTestingServer.cs

@@ -158,6 +158,9 @@ public LiveTestingServer(SwarmConfiguration swarmConfiguration, bool enableOAuth
 				"Telemetry:DisableVersionReporting=true",
 			};
 
+			if (MultiServerClient.UseGraphQL)
+				args.Add("Internal:EnableGraphQL=true");
+
 			swarmArgs = new List<string>();
 			if (swarmConfiguration != null)
 			{

tests/Tgstation.Server.Tests/Live/MultiServerClient.cs

@@ -28,7 +28,7 @@ public MultiServerClient(IRestServerClient restServerClient, IGraphQLServerClien
 		public ValueTask DisposeAsync()
 			=> ValueTaskExtensions.WhenAll(
 				RestClient.DisposeAsync(),
-				GraphQLClient.DisposeAsync());
+				GraphQLClient?.DisposeAsync() ?? ValueTask.CompletedTask);
 
 		public ValueTask Execute(
 			Func<IRestServerClient, ValueTask> restAction,
@@ -48,6 +48,11 @@ public ValueTask Execute(
 			where TGraphQLResult : class
 		{
 			var restTask = restAction(RestClient);
+			if (!UseGraphQL)
+			{
+				return (await restTask, null);
+			}
+
 			var graphQLResult = await GraphQLClient.RunOperation(graphQLAction, cancellationToken);
 
 			graphQLResult.EnsureNoErrors();
@@ -60,6 +65,6 @@ public ValueTask Execute(
 		}
 
 		public ValueTask<IDisposable> Subscribe<TResultData>(Func<IGraphQLClient, IObservable<IOperationResult<TResultData>>> operationExecutor, IObserver<IOperationResult<TResultData>> observer, CancellationToken cancellationToken) where TResultData : class
-			=> GraphQLClient.Subscribe(operationExecutor, observer, cancellationToken);
+			=> GraphQLClient?.Subscribe(operationExecutor, observer, cancellationToken) ?? ValueTask.FromResult<IDisposable>(new CancellationTokenSource());
 	}
 }

tests/Tgstation.Server.Tests/Live/TestLiveServer.cs

@@ -1301,7 +1301,7 @@
 				}
 			}
 			else
 				await internalTask;
 		}
 
 		async Task TestTgsInternal(bool openDreamOnly, CancellationToken hardCancellationToken)
@@ -1930,11 +1930,13 @@
 						password,
 						cancellationToken: cancellationToken);
 					using var cts = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken);
-					graphQLClientTask = graphQLClientFactory.CreateFromLogin(
-						url,
-						username,
-						password,
-						cancellationToken: cts.Token);
+					graphQLClientTask = MultiServerClient.UseGraphQL
+						? graphQLClientFactory.CreateFromLogin(
+							url,
+							username,
+							password,
+							cancellationToken: cts.Token)
+						: ValueTask.FromResult<IAuthenticatedGraphQLServerClient>(null);
 
 					IRestServerClient restClient;
 					try

tests/Tgstation.Server.Tests/Live/UsersTest.cs

@@ -48,6 +48,9 @@ await ValueTaskExtensions.WhenAll(
 
 			await TestPagination(cancellationToken);
 
+			if (!MultiServerClient.UseGraphQL)
+				return;
+
 			Assert.IsFalse(observer.Completed);
 			Assert.AreEqual(0U, observer.ErrorCount);