SQLJames / factorio-server-charts

Helm deployment for Factorio
https://artifacthub.io/packages/helm/factorio-server-charts/factorio-server-charts
The Unlicense
26 stars 29 forks source link

mod list needs to be url encoded #42

Closed seg-falt closed 1 day ago

seg-falt commented 1 year ago

Mod list needs to be URL encoded when there's a space in the mod name.

without encoding: image

with encoding: image

The only relevant difference between the two deployments is the modlist:

without encoding: image

with encoding: image

a urlencode binary is available in a gridsite package(man page), or it can be done through curl dataencode arguements

SQLJames commented 1 year ago

I believe this should just be a modification to the shell script but it's been awhile since I've looked.

I'll take a look this weekend.