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

Added server settings configuration using environment variables. #6

Open
wants to merge 60 commits into
base: master
Choose a base branch
from

Conversation

loginator
Copy link

This PR allows users to generate their ini files using environment variables.
A small C# tool was added to parse ini files and and set the values inside of them to the set environment variables.
I also added a gitignore file to prevent uploading build files and the default volumes.

The readme was changed to help users use the new system.

Dockerfile was updated to build the IniGenerator tool and add it to the main container.

I hope that this is something you are also appreciating and is not going into the wrong direction with the project.
If there are any questions drop a quick comment and I'll try to answer as quick as possible.

@loginator
Copy link
Author

loginator commented Nov 9, 2023

Environment variables can be set as follows:
ASAINI_FileName__SectionName__SettingName=Value
So if you want floating damage numbers you could just set them in the docker compose file using:

- ASAINI_GameUserSettings__ServerSettings__ShowFloatingDamageText=True

@rediculum
Copy link

This is a very important PR for the flexibility of that ASA container. I want to get rid of manually editing INI files. Please proceed asap!

@loginator
Copy link
Author

loginator commented Dec 28, 2023

@Acekorneya shall I resolve the outstanding conflicts? Or is this not the way that is desired we could also go with a small file and add dotnet to the dependencies of the docker image if that'd be better?

@Acekorneya
Copy link
Owner

Do you know how to resolve the current problem of some user having permission issues when they run this docker image under a root user

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants