Hi Daniel, Thank you, I copied changes from your PR and it worked.
Few cases to consider:
I ran it on a azure server and its ip-address:3000 showed the same error with the change. I had to connect it to a subdomain with nginx proxy with APP_DOMAIN as subdomain to make it past through the sign in. So environment variable will be a better choice i think
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
Hi Daniel, Thank you, I copied changes from your PR and it worked.
Few cases to consider:
I ran it on a azure server and its ip-address:3000 showed the same error with the change. I had to connect it to a subdomain with nginx proxy with APP_DOMAIN as subdomain to make it past through the sign in. So environment variable will be a better choice i think
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.