Notes by Greelan:
Some further development of the template that you can merge if appropriate to build out your PR.
Having worked with it a bit more I've concluded that it ain't a particularly AMP-friendly server.
Couple of specific comments:
I've added AdminPassword as a command line argument, but I am not sure that is accepted. Most instructions say to put it in the ServerSettings.ini. Problem is that the default ServerSettings.ini that is created when the server is first started does not include that field. So I tried the solution of having a copy of that ini, with the AdminPassword field added, being downloaded from GitHub and copied across as update stages. But the server overwrites that on first start. And given I don't want the update stages overwriting the existing ServerSettings.ini, because it might include a lot of user configuration, I can't see a resolution for this (ie getting an AdminPassword entry into the standard ServerSettings.ini that is not overwritten by the server and that does not cause AMP to overwrite other settings).
The Server Password manifest entry assumes that it will work with the ServerPassword field that exists by default in ServerSettings.ini. But I've also read that ServerPassword only works in Engine.ini. However, Engine.ini doesn't include that entry by default - which creates the same issue as for AdminPassword.
This brings back the work that was started on https://github.com/CubeCoders/AMPTemplates/pull/3 previously. It has been migrated to use the new generic settings. It currently only works on Windows, but has a baseline started for Proton. Settings have not been added to the GUI yet.
