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

Introduce EMAIL_AUTH=false Environment variable #71

Merged
merged 3 commits into from
Nov 14, 2017

Conversation

justb81
Copy link
Contributor

@justb81 justb81 commented Sep 19, 2017

This PR (which fixes issue #70 ) introduces a new environment variable to disable SMTP authentication.

Just set EMAIL_AUTH to false to ignore smtp authentication. this is useful when operating with internal smarthosts or mailtraps.

- fixes passbolt#70
- allows for setting EMAIL_AUTH=false as environment variable to deactivate SMTP-Auth entirely. This is necessary for internal mailservers / mailtraps / smarthosts, that just expose anonymous port 25.
@CLAassistant
Copy link

CLAassistant commented Sep 19, 2017

CLA assistant check
All committers have signed the CLA.

@dlen dlen self-assigned this Nov 8, 2017
@dlen
Copy link
Member

dlen commented Nov 8, 2017

Hi @justb81 thanks a lot for your contribution to the project!! Apologies for the delay reacting to your PR, we have some other priorities that are consuming our time right now.

PR looks good but some changes might be required, this week some testing time will be allocated and your contribution will be merged ASAP.

Again thanks.

@justb81
Copy link
Contributor Author

justb81 commented Nov 13, 2017

Hi @dlen if you need any changes, don't hesitate contacting me

@dlen
Copy link
Member

dlen commented Nov 14, 2017

@justb81 No changes required, thanks for your contribution 💙

@dlen dlen merged commit f13f374 into passbolt:master Nov 14, 2017
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

Successfully merging this pull request may close these issues.

3 participants