getmeli / meli

Platform for deploying static sites and frontend applications easily. Automatic SSL, deploy previews, reverse proxy, and more.
Other
2.41k stars 97 forks source link

Custom domains don't work for branch deploys #165

Closed queicherius closed 3 years ago

queicherius commented 3 years ago

I have setup DNS records (A) for project.company.io and *.project.company.io, pointing at the Meli IP.

I then added project.company.io into the "Domains" section of the site settings. This now works perfectly for project.company.io itself, pointing towards the main branch.

However, if I call branch.project.company.io, I get a ERR_SSL_PROTOCOL_ERROR, with a "no server TLS configuration available" error.

This is fixable, if I add *.project.company.io to the domains in the site settings, but I feel like this either requires some docs or should happen automatically.

pimartin commented 3 years ago

The original goal for the custom domains was to set the "production" domain for the site while the original goal for the branches was to allow the preview of the "non-prod" versions (beta, demo...). Which is why it is currently not working correctly.

It does however make sense to add the branches to the custom domains in certain cases.

So we're going to:

After this change, the user will need to:

gempain commented 3 years ago

@queicherius so sorry it took this long to get on this, but I'm on it 😄

gempain commented 3 years ago

This has been implemented and will be included in the next release. A toggle under your custom domains will allow you to toggle branches for subdomains. I believe this is a breaking change with your current workaround.

Screen Shot 2021-01-12 at 14 31 15

gempain commented 3 years ago

@queicherius i'm closing, but please let me know if further help is needed, happy to help 😀