Skip to content

Added config options for servers - #144

Open
warriorjacq9-no2fa wants to merge 3 commits into
xandergos:masterfrom
warriorjacq9-no2fa:master
Open

Added config options for servers#144
warriorjacq9-no2fa wants to merge 3 commits into
xandergos:masterfrom
warriorjacq9-no2fa:master

Conversation

@warriorjacq9-no2fa

@warriorjacq9-no2fa warriorjacq9-no2fa commented May 11, 2026

Copy link
Copy Markdown

Added default_scale in terrain-diffusion-mc.properties to allow dedicated servers to change the world scale, and explorer.address to allow for exposing the terrain explorer

@warriorjacq9-no2fa warriorjacq9-no2fa changed the title Add default_scale option to config for servers Added config options for servers May 12, 2026
@ThatDamnWittyWhizHard ThatDamnWittyWhizHard added type: enhancement Improvement to an existing feature. priority: medium Important, but not blocking. Should be handled after critical and high-priority tasks. status: needs testing Implemented or fixed, but still needs testing. env: dev Issue related to the development environment. labels May 12, 2026
@ThatDamnWittyWhizHard ThatDamnWittyWhizHard added this to the 2.3.0 milestone May 12, 2026
@xandergos

Copy link
Copy Markdown
Owner

The default scale change is fine, though im not sure its a good idea to always expose /terrain-explorer to users. Its pretty easy to abuse server resources with it. I hadn't considered this when creating the mod; it should probably be configurable.

@ThatDamnWittyWhizHard

Copy link
Copy Markdown
Collaborator

This is what I thought too. It looks like they managed to make it work in this issue : Server Configuration

So maybe better documentation would be enough?

@ThatDamnWittyWhizHard ThatDamnWittyWhizHard modified the milestones: 2.3.0, 2.4.0 May 13, 2026
@warriorjacq9-no2fa

warriorjacq9-no2fa commented May 13, 2026

Copy link
Copy Markdown
Author

I don't think the average user would want to manually edit NBT tags (I don't, that's why I made this modification). As for /td-explore safety, maybe a warning in the config? It's still bound to localhost by default

@EmeraldGltch

Copy link
Copy Markdown

I don't think the average user would want to manually edit NBT tags

Seconding this, my hacky fix doesn't seem to work on 1.21.1 in my personal testing.

@xandergos

Copy link
Copy Markdown
Owner

Yeah that's true I forgot that port forwarding would still have to be done manually. So looks good then.

@ThatDamnWittyWhizHard

Copy link
Copy Markdown
Collaborator

Could a client map explorer be a better option for /td-explore when players are in the solo, customization tab or multiplayer?

@ThatDamnWittyWhizHard

Copy link
Copy Markdown
Collaborator

Like so we just need a property to allow dedicated servers to change the world scale properly.

@warriorjacq9-no2fa

warriorjacq9-no2fa commented May 17, 2026

Copy link
Copy Markdown
Author

You can do a partial merge with a local clone and git cherry-pick if you need to. The last commit is the one that adds binding td-explore. Alternatively I can roll that commit back if needed

@ThatDamnWittyWhizHard

Copy link
Copy Markdown
Collaborator

I’ll keep your PR open for now. Once 2.3.0 is stable I’ll merge the relevant parts of your PR.

@ThatDamnWittyWhizHard ThatDamnWittyWhizHard added status: blocked Blocked by a dependency, decision or external issue. and removed status: needs testing Implemented or fixed, but still needs testing. labels May 18, 2026
@PhillipSteph

Copy link
Copy Markdown

How would one set the terrain generation to terrain_diffusion in the standard server.properties or else?

@warriorjacq9-no2fa

Copy link
Copy Markdown
Author

Set level-type to terrain-diffusion-mc:terrain_diffusion I think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

env: dev Issue related to the development environment. priority: medium Important, but not blocking. Should be handled after critical and high-priority tasks. status: blocked Blocked by a dependency, decision or external issue. type: enhancement Improvement to an existing feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants