Skip to content
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

fix(minecraft): set server onlineMode to false when behind proxy #4430

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

tyriis
Copy link
Owner

@tyriis tyriis commented Jan 31, 2025

No description provided.

@tyriis-automation
Copy link
Contributor

--- kubernetes/talos-flux/apps/gaming-public/minecraft-java/playground-world Kustomization: flux-system/minecraft-public-playground-world HelmRelease: gaming-public/minecraft-public-playground-world

+++ kubernetes/talos-flux/apps/gaming-public/minecraft-java/playground-world Kustomization: flux-system/minecraft-public-playground-world HelmRelease: gaming-public/minecraft-public-playground-world

@@ -65,13 +65,13 @@

       gameMode: creative
       generateStructures: false
       levelType: FLAT
       maxPlayers: 20
       memory: 1024M
       motd: techicraft playground
-      onlineMode: true
+      onlineMode: false
       ops: ''
       pvp: false
       rcon:
         enabled: false
       spawnAnimals: false
       spawnMonsters: false

@tyriis-automation
Copy link
Contributor

--- HelmRelease: gaming-public/minecraft-public-playground-world Deployment: gaming-public/minecraft-public-playground-world-minecraft

+++ HelmRelease: gaming-public/minecraft-public-playground-world Deployment: gaming-public/minecraft-public-playground-world-minecraft

@@ -94,13 +94,13 @@

           value: 'false'
         - name: LEVEL_TYPE
           value: FLAT
         - name: LEVEL
           value: playground
         - name: ONLINE_MODE
-          value: 'true'
+          value: 'false'
         - name: MEMORY
           value: 1024M
         - name: ENABLE_RCON
           value: 'false'
         - name: CF_API_KEY
           valueFrom:

@tyriis-automation
Copy link
Contributor

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 1 0 0.01s
✅ REPOSITORY gitleaks yes no 3.17s
✅ YAML prettier 1 0 0.3s
✅ YAML yamllint 1 0 0.34s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@tyriis tyriis merged commit a200b07 into main Jan 31, 2025
16 checks passed
@tyriis tyriis deleted the feature/minecraft-public branch January 31, 2025 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant