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

Support hostname when setting up a route+domain service #27

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

mogul
Copy link
Contributor

@mogul mogul commented Jun 12, 2023

For example, one might want their app mapped to the FQDN "app.fac.gov", not just "fac.gov". 😉

For example, an app may be mapped to the domain "app.thedomain.gov",
not just "thedomain.gov".
@mogul mogul requested a review from rahearn June 12, 2023 23:59
@mogul
Copy link
Contributor Author

mogul commented Jun 13, 2023

I'm about 90% sure that this will work... It's hard to test easily without passing something all the way through our pipeline.

Copy link

@akf akf left a comment

Choose a reason for hiding this comment

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

Looks reasonable.

Copy link
Contributor

@pauldoomgov pauldoomgov left a comment

Choose a reason for hiding this comment

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

This looks good to me, for what it is worth!

@pauldoomgov pauldoomgov merged commit 7db01d5 into main Jun 13, 2023
@pauldoomgov pauldoomgov deleted the support-hostname-on-domain-service branch June 13, 2023 01:06
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.

3 participants