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

Mailhog required for some commands #809

Closed
danepowell opened this issue Dec 19, 2016 · 1 comment
Closed

Mailhog required for some commands #809

danepowell opened this issue Dec 19, 2016 · 1 comment
Labels
Enhancement A feature or feature request

Comments

@danepowell
Copy link
Contributor

danepowell commented Dec 19, 2016

My system information:

  • BLT version: 8.5.2
  • LAMP stack is DrupalVM 3.5.2

When I run this command:

blt local:refresh

I get the following output:

sh: 1: /opt/mailhog/mhsendmail: not found

@geerlingguy says "we should either choose mailhog in installed_extras by default, or switch the PHP mail command back to what it was"

I confirmed that adding mailhog to installed_extras fixes it.

@geerlingguy
Copy link
Contributor

See #810 — this takes the former approach, and matches Drupal VM's default configuration. IMO this is better because having a mail catching utility running locally means no matter what, unless you explicitly override it, developers can't accidentally shoot out hundreds of emails to real email addresses :)

@grasmash grasmash added the Enhancement A feature or feature request label Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request
Projects
None yet
Development

No branches or pull requests

3 participants