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

feat: Expose proxy hosted zone id #30

Merged
merged 1 commit into from
Jan 13, 2021
Merged

feat: Expose proxy hosted zone id #30

merged 1 commit into from
Jan 13, 2021

Conversation

maael
Copy link
Contributor

@maael maael commented Jan 13, 2021

Thanks for the project, and your time spent on it!

These small changes aim to allow more complicated aliasing than currently possible with just the name exposed. Having both the domain name and hosted zone id will allow us to punch it into the alias on our own controlled aws_route53_record resource directly for a more custom approach to custom domains - as it needs both these values.

The main reason for this is that we currently have different providers in different regions for our infrastructure and managing dns/route53, which as the providers differ means we can't use the currently available approach to custom domains.

This allows more complicated aliasing that currently possible with just the name exposed. Having both the domain name and hosted zone id will allow us to punch it into the alias on our own external `aws_route53_record` for a more custom approach to custom domains. The main reason for this is that we currently have different providers in different regions for our infrastructure and managing dns/route53, which as the providers differ means we can't use the currently available approach to custom domains.
Copy link
Member

@ofhouse ofhouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

The initial output values of the module are very limited (Since they only reflect our internal usage), but we are always open to add more values to support other use cases. 👍

@ofhouse ofhouse merged commit e42e9a2 into milliHQ:main Jan 13, 2021
@ofhouse
Copy link
Member

ofhouse commented Jan 13, 2021

Now released as v0.5.1 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants