web icon indicating copy to clipboard operation
web copied to clipboard

unnecessary function call (perhaps)

Open ImOver00 opened this issue 4 years ago • 2 comments

https://github.com/openmultiplayer/web/blob/3eac1c3f5f5801321bef2b6e533566af004e5440/server/main.go#L15 importing autoload will build it even tho its not used but it will at least execute the init function. Meanwhile the "init" function already does godotenv.Load(). so why are you importing autoload and doing that at the same time? Shouldn't you stick to ONLY one of these?

ImOver00 avatar Feb 02 '22 16:02 ImOver00

yeah it probably should be one or the other, can't remember why but I think it was load order. experiment with removing one and seeing if the app still works properly

Southclaws avatar Feb 03 '22 11:02 Southclaws

thanks, will do soon

ImOver00 avatar Feb 03 '22 11:02 ImOver00