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

Custom domains don't work for branch deploys #165

Closed
queicherius opened this issue Dec 15, 2020 · 4 comments
Closed

Custom domains don't work for branch deploys #165

queicherius opened this issue Dec 15, 2020 · 4 comments
Labels
enhancement Improvement of an existing feature
Projects

Comments

@queicherius
Copy link

queicherius commented Dec 15, 2020

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
Copy link
Contributor

pimartin commented Dec 15, 2020

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:

  • add a toggle to enable/disable branches on a custom domain
  • if enabled:
    • add the subdomains to the SSL configuration
    • add the subdomains to the routes (this happens unconditionally at this time)
  • (add validation on the domain name setting to prevent the use of wildcard domains?)
  • improve/update the documentation to reflect this change

After this change, the user will need to:

  • specify its domains without wildcard in the settings (project.company.io)
  • enable the branches for the domain
  • add a wildcard entry in the DNS zone (*.project.company.io)

@pimartin pimartin added the enhancement Improvement of an existing feature label Dec 15, 2020
@gempain gempain self-assigned this Dec 17, 2020
@gempain gempain added this to the v1.0.0-beta.7 milestone Dec 17, 2020
@gempain gempain added this to To do in main Dec 18, 2020
@gempain gempain removed this from the v1.0.0-beta.7 milestone Dec 18, 2020
@gempain gempain removed their assignment Jan 7, 2021
@gempain gempain moved this from To do to In progress in main Jan 12, 2021
@gempain
Copy link
Contributor

gempain commented Jan 12, 2021

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

@gempain
Copy link
Contributor

gempain commented Jan 12, 2021

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 pushed a commit that referenced this issue Jan 13, 2021
# [1.0.0-beta.14](v1.0.0-beta.13...v1.0.0-beta.14) (2021-01-13)

### Bug Fixes

* missing await in org list ([ea5c456](ea5c456))
* missing await in org member serializer ([7617d86](7617d86))
* sites not serialized properly ([eb8ee20](eb8ee20))
* team sites not serialized properly ([e215f0e](e215f0e))

### Features

* expose branches on custom domains [#165](#165) ([46f3de3](46f3de3))
* logos ([#183](#183)) ([0074e42](0074e42))
* show logo to invites and org members ([c5791fe](c5791fe))
@gempain
Copy link
Contributor

gempain commented Jan 15, 2021

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

@gempain gempain closed this as completed Jan 15, 2021
@gempain gempain moved this from In progress to Done in main Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement of an existing feature
Projects
main
Done
Development

No branches or pull requests

3 participants