Yes, I think the environment variable is a better solution - the docker-compose file was originally written to support local development using docker (I think we didn't have enough people using it locally to notice the issue you saw with the login submissions failing).
I've merged the changes into the main branch (so you shouldn't need to worry about re-applying that patch going forward). However, do feel free to add any environment variables you need to the compose file or pass them at startup.
For further actions, you may consider blocking this person and/or reporting abuse
Yes, I think the environment variable is a better solution - the docker-compose file was originally written to support local development using docker (I think we didn't have enough people using it locally to notice the issue you saw with the login submissions failing).
I've merged the changes into the main branch (so you shouldn't need to worry about re-applying that patch going forward). However, do feel free to add any environment variables you need to the compose file or pass them at startup.