-
-
Notifications
You must be signed in to change notification settings - Fork 595
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
feat: replace MailHog with Mailpit, provide global router port configuration, fixes #4827, fixes #5220, for #4701 #5313
Conversation
Download the artifacts for this pull request:
See Testing a PR |
cd496af
to
318429d
Compare
|
||
# host_mailhog_port: "8025" | ||
# The mailhog port is not normally bound on the host at all, instead being routed | ||
# host_mailpit_port: "8025" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is probably the least used configuration because this is the first time I see it.
There is no documentation for it (except for DDEV_HOST_MAILPIT_PORT
) and no such option in ddev config
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added the doc. Agreed it's mighty obscure.
Confirmed, works OOTB on Drupal 10. TestService
$ ddev -v
ddev version v1.22.1-58-g3a427fd30
mkdir my-drupal10-site
cd my-drupal10-site
ddev config --project-type=drupal10 --docroot=web --create-docroot
ddev start
ddev composer create drupal/recommended-project
ddev composer require drush/drush
ddev drush site:install --account-name=admin --account-pass=admin -y
ddev drush uli
ddev launch
Status
|
3a427fd
to
7f53627
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found some minor capitalization issues, but otherwise looks good to me!
I changed mailpit-port to mailpit-http-port, etc. |
Co-authored-by: Matt Stein <m@ttste.in>
…lthough deprecated
4d5fa38
to
81418fc
Compare
The Issue
How This PR Solves The Issue
All I wanted to do was retire the fork. But it still didn't have a decent arm64 download. So I just kept going. Used up my day!
Manual Testing Instructions
Use
ddev launch -m
as usualAutomated Testing Overview
Added TestMailpit - there was nothing previously
Release concerns
This invalidates https://github.com/tyler36/ddev-mailpit although it's possible it would still be compatible.