Open tyler36 opened 1 month ago
Haven't had a chance to test it yet.
OK. Not sure what changes you would like here though.
There is no way to change it here, as it needs to be changed in DDEV.
But I'm not sure why it needs to be changed. Is it ddev mailpit
that needs to be changed? What is the goal?
Consistency, mostly.
EG. Xdebug port:
9000
as xdebug port, but xdebug is on 9003
. So this is confusing.Xdebug has been on 9003 for several years, since before gitpod support. I'm wondering how it got that?
If you can live with mailpit listening on 8027, it's OK there. People shouldn't have to know where these ports are.
Sorry, I got confused between your two PRs
So where are we with this?
I think it's good.
Can you also add more ports from the official add-ons?:
Added ports but have not tested.
@stasadev Oops. Forgot the HTTP ports. Good catch!
The Issue
I was debugging Mailpit issues in Gitpod when I noticed many of the ports are outdated.
How This PR Solves The Issue
This PR
This PR does NOT update the Mailpit port. See #40
Manual Testing Instructions
Automated Testing Overview
Related Issue Link(s)
Release/Deployment Notes