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

Cannot send message without a sender address #75

Closed
aronmal opened this issue May 11, 2022 · 2 comments
Closed

Cannot send message without a sender address #75

aronmal opened this issue May 11, 2022 · 2 comments

Comments

@aronmal
Copy link
Contributor

aronmal commented May 11, 2022

Is your feature request related to a problem? Please describe.
An error message is showing up if you click "Forgot your password? Reset it", then type in your email, click "Send password reset link" and be surprised with the error.

Describe the solution you'd like
Is see email related setting in the app.json file, but I did not found this file in my docker volume folder and there does not seem to be docker environment variables to adjust these. So if it email delivery is already implemented, the settings should be adjustable for individual users.

Additional context
grafik

@aronmal
Copy link
Contributor Author

aronmal commented May 11, 2022

OK, if I correct my self, I found this:

As described in the Docker doc you can modify and use the docker-compose file or start 2FAuth with custom env variables, like so:

docker run --rm -d  -p 8000:8000/tcp \
MAIL_DRIVER=smtp \
MAIL_HOST=SSL0.OVH.NET \
MAIL_PORT=465 \
MAIL_USERNAME=myName@domain.org \
MAIL_PASSWORD=MyEmailProviderPwd \
MAIL_ENCRYPTION=ssl \
MAIL_FROM_NAME="MyName" \
MAIL_FROM_ADDRESS=myName@domain.org \
2fauth/2fauth:latest

As seen in #50 (comment)

But it is no where documented. So please update docs somewhere and the dockerhub description please.

@aronmal
Copy link
Contributor Author

aronmal commented May 11, 2022

Is the .env.example still a thing?

Or is this:

You can set environment variables available (see the .env.example) with -e, for example -e APP_NAME=2FAuth.

the way to got to set the variables, because of this:

What Bubka/2Fauth-Docker qdm12/2Fauth Final Why
Read env from files yes no no there is not much of a point since the env is populated with content of secret files. Correct me if I'm wrong though
Env Use .env Use env variables injected by Docker Use env variables injected by Docker Simpler to use for Docker users. They can always use a .env on their host to populate their env with Docker

Either way, the documentation should be updated to be more clear.

@aronmal aronmal closed this as completed May 12, 2022
Bubka pushed a commit that referenced this issue May 13, 2022
* Fix recommendation for Bubka #68 in docs

Adding a recommendation to use the AUTHENTICATION_GUARD environment variable as a fix for issue #68.

* Mail Settings in docker README.md

#75
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

No branches or pull requests

1 participant