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.

Sign up for GitHub

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

Jump to bottom

Update .env.example: remove MAIL_FROM; add MAIL_VERIFY_SSL_PEER #247

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Update .env.example: remove MAIL_FROM; add MAIL_VERIFY_SSL_PEER #247

merged 1 commit into from
Dec 5, 2023

Conversation

rouilj
Copy link
Contributor

@rouilj rouilj commented Nov 23, 2023

@Bubka
Copy link
Owner

Bubka commented Nov 23, 2023

Thanks for you PR.

Would be great if you could make the same change to docker/docker-compose.yml and Dockerfile.

@rouilj
Copy link
Contributor Author

rouilj commented Nov 23, 2023

I added pull #248 and pull #249 to remove MAIL_FROM. I didn't add MAIL_VERIFY_SSL_PEER as
I am not sure what the default value should be.

@Bubka
Copy link
Owner

Bubka commented Nov 23, 2023

Adding new commits to this PR would have been more convenient 😅
Nevermind, thx for your contribution.

@Bubka Bubka changed the base branch from master to dev December 5, 2023 16:02
@Bubka Bubka added this to the v5.0 milestone Dec 5, 2023
@Bubka Bubka merged commit 85506c7 into Bubka:dev Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
2FAuth backlog
Status: Released
Milestone
v5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@rouilj @Bubka