-
-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add OpenDream support and SessionId
#1666
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Big |
Cyberboss
force-pushed
the
OpenDream
branch
2 times, most recently
from
October 9, 2023 19:16
ca76ce7
to
45a4498
Compare
Cyberboss
force-pushed
the
OpenDream
branch
2 times, most recently
from
October 9, 2023 19:31
d4e01a5
to
eaa3bad
Compare
Done squashing commits
Drop support for custom OD repositories, hopefully fix custom installs. Need to go back to the git strat because of submodules
Also, report checkout progress
…D repo manager Various other things to get it compiling
src/Tgstation.Server.Host/Components/Engine/ByondInstallerBase.cs
Dismissed
Show dismissed
Hide dismissed
src/Tgstation.Server.Host/Components/Engine/EngineManager.cs
Dismissed
Show dismissed
Hide dismissed
src/Tgstation.Server.Host/Components/Engine/ByondInstallerBase.cs
Dismissed
Show dismissed
Hide dismissed
src/Tgstation.Server.Host/Components/Engine/ByondInstallerBase.cs
Dismissed
Show dismissed
Hide dismissed
src/Tgstation.Server.Host/Components/Engine/ByondInstallerBase.cs
Dismissed
Show dismissed
Hide dismissed
- Make use of `world.opendream_topic_port` in DMAPI if present. Send in initial bridge request. - Implement compiler and server arguments for `OpenDreamInstallation`. - Deprecate bridge port update requests. - Prevent symlink swapping if not running a BYOND engine.
- Fix race condition with updating launch parameters. - Add additional logging. - Add integration tests
Also fixes logic error in `AdvancedWatchdog.HandleNewDmbAvailable` in the `EngineType.OpenDream` case.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Area: Compiler
With regard to managing the deployment process
Area: DMAPI
Communication between TGS and DM
Area: Engine
With regard to managing engine installations
Area: Watchdog
With regard to managing DreamDaemon servers
Breaking API Change
Requires a major API version bump
CI/CD
IT'S NOT A PHASE MOM, IT'S A REAL SPECIALIZATION
Client
Issue with the .NET client library
Component Issue
Issue regarding the service components
Configuration
Regarding the server setup JSON
Controller Issue
Issue regarding the HTTP request controllers
Feature
New functionality
Fix
Fixes incorrect functionality
Logging
Adding, removing, or lack of informational log output
Migration
Requires or performs a database migration
Refactor
Refactor functionality for future improvements
REST
The JSON REST API for server control
Watchdog: Basic
Issue with the BasicWatchdog
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🆑
TGS now supports running DreamMaker coded games with OpenDream, the open source reimplementation of BYOND. Note that TGS requires the dotnet SDK version supported by the OpenDream version you'd like to use alongside it's normal ASP.NET Core runtime to build it.
The repository cloning job now reports checkout progress.
Fixed rare DreamDaemon startup failure that may occur due to a lingering socket.
Event scripts containing "Byond" in their names have been updated to use "Engine" instead. Old names will continue function.
Socket bind tests will now retry for up to 5s before failing with an AddressInUse exception.
Added a workaround for a potential hang while recording process output.
/:cl:
🆑 HTTP API
There has been a major version bump to API version 10.
The entire
/Byond
route has been removed.Added new
/Engine
route and associated models. This has similar functions to the old/Byond
route while being engine agnostic so as to support OpenDream.InstancePermissionSet.byondRights
has been renamed toInstancePermissionSet.engineRights
.CompileJobResponse.byondVersion
changed to newCompileJobResponse.engineVersion
model.Added
DreamDaemonResponse.sessionId
which is a unique ID for the current server execution./:cl:
🆑 Nuget: Api
Performed all API 10 model changes.
/:cl:
🆑 Nuget: Client
Performed all API 10 client changes.
/:cl:
🆑 DreamMaker API
Various event names changed to be engine agnostic.
Corrected documentation that stated a
/datum/tgs_version
engine version would be returned to properly say they will be stringified.Added enums and function for retrieving the current engine version.
/:cl:
Closes #1577
Required:
Mixed line endings error in OD OpenDreamProject/OpenDream#1492/file
not a string OpenDreamProject/OpenDream#1497/world.Topic()
leads to an invalid response OpenDreamProject/OpenDream#1396/Engine
route?