diff --git a/build/ControlPanelVersion.props b/build/ControlPanelVersion.props index f13fb24260..dc9dc3ae40 100644 --- a/build/ControlPanelVersion.props +++ b/build/ControlPanelVersion.props @@ -1,6 +1,6 @@ - 2.1.3 + 2.1.4 diff --git a/build/Version.props b/build/Version.props index 834e9efe77..1add9090b6 100644 --- a/build/Version.props +++ b/build/Version.props @@ -7,7 +7,7 @@ 3.0.0 8.3.0 9.2.0 - 6.0.1 + 6.0.2 5.3.0 1.1.1 1.2.0 diff --git a/src/DMAPI/tgs.dm b/src/DMAPI/tgs.dm index de9533dc32..f89c696020 100644 --- a/src/DMAPI/tgs.dm +++ b/src/DMAPI/tgs.dm @@ -1,6 +1,6 @@ // tgstation-server DMAPI -#define TGS_DMAPI_VERSION "6.0.1" +#define TGS_DMAPI_VERSION "6.0.2" // All functions and datums outside this document are subject to change with any version and should not be relied on. @@ -101,7 +101,7 @@ // #define TGS_EVENT_WORLD_END_PROCESS 19 // #define TGS_EVENT_WORLD_REBOOT 20 /// Watchdog event when TgsInitializationComplete() is called. No parameters. - #define TGS_EVENT_WORLD_PRIME 21 +#define TGS_EVENT_WORLD_PRIME 21 // OTHER ENUMS diff --git a/src/DMAPI/tgs/v3210/api.dm b/src/DMAPI/tgs/v3210/api.dm index 5b41e6a89d..245381b93e 100644 --- a/src/DMAPI/tgs/v3210/api.dm +++ b/src/DMAPI/tgs/v3210/api.dm @@ -160,7 +160,7 @@ var/datum/tgs_revision_information/test_merge/tm = new tm.number = text2num(I) var/list/entry = json[I] - tm.pull_request_commit = entry["commit"] + tm.head_commit = entry["commit"] tm.author = entry["author"] tm.title = entry["title"] . += tm diff --git a/src/Tgstation.Server.Host/Controllers/ApiController.cs b/src/Tgstation.Server.Host/Controllers/ApiController.cs index 0c37a75142..beaae1e95d 100644 --- a/src/Tgstation.Server.Host/Controllers/ApiController.cs +++ b/src/Tgstation.Server.Host/Controllers/ApiController.cs @@ -182,9 +182,6 @@ namespace Tgstation.Server.Host.Controllers if (headersException.MissingOrMalformedHeaders.HasFlag(HeaderTypes.Accept)) return StatusCode(HttpStatusCode.NotAcceptable, errorMessage); - if (headersException.MissingOrMalformedHeaders == HeaderTypes.Authorization) - return Unauthorized(errorMessage); - return BadRequest(errorMessage); } diff --git a/tests/DMAPI/BasicOperation/Config.dm b/tests/DMAPI/BasicOperation/Config.dm index 1fff7f2a61..3cd3f332de 100644 --- a/tests/DMAPI/BasicOperation/Config.dm +++ b/tests/DMAPI/BasicOperation/Config.dm @@ -9,3 +9,4 @@ #define TGS_ERROR_LOG(message) world.log << "Err: [##message]" #define TGS_NOTIFY_ADMINS(event) #define TGS_CLIENT_COUNT 0 +#define TGS_V3_API diff --git a/tests/DMAPI/BasicOperation/Test.dm b/tests/DMAPI/BasicOperation/Test.dm index 23dbc6295c..d0119e52fc 100644 --- a/tests/DMAPI/BasicOperation/Test.dm +++ b/tests/DMAPI/BasicOperation/Test.dm @@ -17,7 +17,7 @@ var/list/world_params = params2list(world.params) if(!("test" in world_params) || world_params["test"] != "bababooey") text2file("Expected parameter test=bababooey but did not receive", "test_fail_reason.txt") - + world.log << "sleep2" sleep(150) world.log << "Terminating..." diff --git a/tests/Tgstation.Server.Tests/Instance/WatchdogTest.cs b/tests/Tgstation.Server.Tests/Instance/WatchdogTest.cs index 5cd9871d7b..80b88a3bde 100644 --- a/tests/Tgstation.Server.Tests/Instance/WatchdogTest.cs +++ b/tests/Tgstation.Server.Tests/Instance/WatchdogTest.cs @@ -143,7 +143,7 @@ namespace Tgstation.Server.Tests.Instance Assert.IsNotNull(newerCompileJob); Assert.AreNotEqual(initialCompileJob.Id, newerCompileJob.Id); - Assert.AreEqual(DreamDaemonSecurity.Safe, newerCompileJob.MinimumSecurityLevel); + Assert.AreEqual(DreamDaemonSecurity.Trusted, newerCompileJob.MinimumSecurityLevel); Assert.AreEqual(DMApiConstants.InteropVersion, daemonStatus.StagedCompileJob.DMApiVersion); await instanceClient.DreamDaemon.Shutdown(cancellationToken); } @@ -157,13 +157,13 @@ namespace Tgstation.Server.Tests.Instance AdditionalParameters = "test=bababooey" }, cancellationToken); Assert.AreEqual("test=bababooey", daemonStatus.AdditionalParameters); - daemonStatus = await DeployTestDme("BasicOperation/basic_operation_test", DreamDaemonSecurity.Ultrasafe, true, cancellationToken); + daemonStatus = await DeployTestDme("BasicOperation/basic_operation_test", DreamDaemonSecurity.Trusted, true, cancellationToken); Assert.AreEqual(WatchdogStatus.Offline, daemonStatus.Status.Value); Assert.IsNotNull(daemonStatus.ActiveCompileJob); Assert.IsNull(daemonStatus.StagedCompileJob); Assert.AreEqual(DMApiConstants.InteropVersion, daemonStatus.ActiveCompileJob.DMApiVersion); - Assert.AreEqual(DreamDaemonSecurity.Safe, daemonStatus.ActiveCompileJob.MinimumSecurityLevel); + Assert.AreEqual(DreamDaemonSecurity.Trusted, daemonStatus.ActiveCompileJob.MinimumSecurityLevel); Job startJob; if (new PlatformIdentifier().IsWindows) // Can't get address reuse to trigger on linux for some reason diff --git a/tests/Tgstation.Server.Tests/RawRequestTests.cs b/tests/Tgstation.Server.Tests/RawRequestTests.cs index df9bad0018..a9e308e73b 100644 --- a/tests/Tgstation.Server.Tests/RawRequestTests.cs +++ b/tests/Tgstation.Server.Tests/RawRequestTests.cs @@ -136,6 +136,20 @@ namespace Tgstation.Server.Tests Assert.AreEqual(ErrorCode.InstanceHeaderRequired, message.ErrorCode); } + using (var request = new HttpRequestMessage(HttpMethod.Get, url.ToString())) + { + request.Headers.Accept.Clear(); + request.Headers.UserAgent.Add(new ProductInfoHeaderValue("RootTest", "1.0.0")); + request.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue(MediaTypeNames.Application.Json)); + request.Headers.Add(ApiHeaders.ApiVersionHeader, "Tgstation.Server.Api/" + ApiHeaders.Version); + request.Headers.Authorization = new AuthenticationHeaderValue(ApiHeaders.BearerAuthenticationScheme.ToLower(), token); + using var response = await httpClient.SendAsync(request, cancellationToken); + Assert.AreEqual(HttpStatusCode.BadRequest, response.StatusCode); + var content = await response.Content.ReadAsStringAsync(); + var message = JsonConvert.DeserializeObject(content); + Assert.AreEqual(ErrorCode.BadHeaders, message.ErrorCode); + } + using (var request = new HttpRequestMessage(HttpMethod.Post, url.ToString())) { request.Headers.Accept.Clear();