LinuxGSM icon indicating copy to clipboard operation
LinuxGSM copied to clipboard

feat(newserver): add Velocity Proxy

Open h3o66 opened this issue 2 years ago • 6 comments

Description

Please include a summary of the change and which issues are fixed.

Fixes #3436

Type of change

  • [ ] Bug fix (a change which fixes an issue).
  • [ ] New feature (change which adds functionality).
  • [x] New Server (new server added).
  • [ ] Refactor (restructures existing code).
  • [ ] Comment update (typo, spelling, explanation, examples, etc).

Checklist

PR will not be merged until all steps are complete.

  • [x] This pull request links to an issue.
  • [x] This pull request uses the develop branch as its base.
  • [x] This pull request Subject follows the Conventional Commits standard.
  • [x] This code follows the style guidelines of this project.
  • [x] I have performed a self-review of my code.
  • [x] I have checked that this code is commented where required.
  • [x] I have provided a detailed with enough description of this PR.
  • [x] I have checked If documentation needs updating.

Documentation

If documentation does need updating either update it by creating a PR (preferred) or request a documentation update.

  • User docs: https://github.com/GameServerManagers/LinuxGSM-Docs
  • Dev docs: https://github.com/GameServerManagers/LinuxGSM-Dev-Docs

Thank you for your Pull Request!

h3o66 avatar Jun 26 '22 21:06 h3o66

TBD: check if upload of the default config needed or not TBD2: create new shutdown command "shutdown" to send to console

h3o66 avatar Jun 26 '22 22:06 h3o66

Updated to create the new stop command "shutdown"

h3o66 avatar Jun 27 '22 19:06 h3o66

Added port message, config parsing and dependency for java 11

h3o66 avatar Jun 27 '22 19:06 h3o66

Added a bit more of config parsing. I did not see the need to add the config to our repo, at is created at the first start.

The current latest version does not seem to be a stable one, that is the only curlprint for this.

h3o66 avatar Jun 27 '22 20:06 h3o66

Is this ready to merge?

dgibbs64 avatar Aug 23 '22 20:08 dgibbs64

Is this ready to merge?

From my side this is ready. Would be good if someone else can test this before this is released.

h3o66 avatar Aug 24 '22 13:08 h3o66

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

github-actions[bot] avatar Sep 25 '23 00:09 github-actions[bot]