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

Add ability to send emails to private domain owners #50

Merged
merged 18 commits into from
May 24, 2018
Merged

Conversation

maciej-szlosarczyk
Copy link
Contributor

@maciej-szlosarczyk maciej-szlosarczyk commented May 22, 2018

Can be merged already

The following new variables are needed on staging and then production:

# Variables for mailer
mailer_from_address: 'no-reply@internet.ee'
mailer_host: 'example.com'
smtp_address: 'server-hostname'
smtp_port: '25' # 587, 465
smtp_user_name: 'login'
smtp_password: 'pw/key'
# If you need to specify a HELO domain, you can do it here.
smtp_domain: '' # 'domain for HELO checking'
# Use "none" only when for a self-signed and/or wildcard certificate
smtp_openssl_verify_mode: 'peer' # 'none', 'peer', 'client_once','fail_if_no_peer_cert'
# Detects if STARTTLS is enabled in your SMTP server and starts to use it. Defaults to true.
# Set this to false if there is a problem with your server certificate that you cannot resolve.
smtp_enable_starttls_auto: 'true' # 'false'
# If your mail server requires authentication, please change.
smtp_authentication: 'plain' # 'plain', 'login', 'cram_md5'

We most likely can copy the values from registry, except for mailer_from_address and mailer_host, which are new variables. mailer_host is very important to set up properly, otherwise the links in emails direct people to wrong places.

@maciej-szlosarczyk maciej-szlosarczyk force-pushed the registry-824 branch 2 times, most recently from 70a5923 to f38bb05 Compare May 23, 2018 06:48
@maciej-szlosarczyk
Copy link
Contributor Author

JSON for private person now includes additional field, this info should be communicated to voog:

json.contact_form_link

Other than that, this is up for testing, please pay attention to text as there are some gaps in translation, and if you find any translation that is off/missing, make a comment here. I'll add them accordingly.

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.

2 participants