Hey @saley89! Thanks for opening your first issue. We appreciate your contribution and welcome you to our community! We are glad to have you here and to have your input on Contour. You can also join us on our mailing list and in our channel in the Kubernetes Slack Workspace

Hi @saley89, I haven't formed an opinion yet, but found that Daniel Stenberg (of curl fame) has recorded some details about trailing dot vs HTTP here https://daniel.haxx.se/blog/2022/05/12/a-tale-of-a-trailing-dot/. Interesting read! :)

@tsaarni sure but just to be clear curl was used merely to illustrate the point above. We are seeing this failure from other clients too of course.

Sure 👍 I also did not mean it was related to curl, just that there were surprising complications. It wonder if those have been reason why Envoy defaults to distinguishing between the two.

The Contour project currently lacks enough contributors to adequately respond to all Issues.

This bot triages Issues according to the following rules:

• After 60d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, the Issue is closed

You can:

• Mark this Issue as fresh by commenting
• Close this Issue
• Offer to help out with triage

Please send feedback to the #contour channel in the Kubernetes Slack

The Contour project currently lacks enough contributors to adequately respond to all Issues.

This bot triages Issues according to the following rules:

• After 60d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, the Issue is closed

You can:

• Mark this Issue as fresh by commenting
• Close this Issue
• Offer to help out with triage

Please send feedback to the #contour channel in the Kubernetes Slack

Is there any scope for reopening this issue please? In some integration testing within our company the lack of support for dot domains is causing issues meaning they are having to call the host with out it.

As per the original description, envoy itself supports what is required. We simply need a mechanism to be able to set strip_trailing_host_dot to true and we would have the behaviour we need.

Is it possible to set an envoy configuration value like this today or does explicit support need building into Contour?

Is it possible to set an envoy configuration value like this today or does explicit support need building into Contour?

Support in Contour is required. The field is part of HttpConnectionManager structure, which Contour sends to Envoy via gRPC. Changing the default seems to have a risk of side effects, but there could be (global) configuration parameter to control it.

That's right @tsaarni I think we just want to have a single configuration within Contour that if set to true, set's strip_trailing_host_dot to true on every listener configuration that's generated.

Given the specification I linked to in the description, I'm unclear why the default from Envoy is even set to false?

I did not find reasoning why false is set by default but I understood it could cause problems. If possible, you could try if it works for you with this minimal change to Contour code:

diff --git a/internal/envoy/v3/listener.go b/internal/envoy/v3/listener.go
index 213a71af..e9677a02 100644
--- a/internal/envoy/v3/listener.go
+++ b/internal/envoy/v3/listener.go
@@ -519,6 +519,7 @@ func (b *httpConnectionManagerBuilder) Get() *envoy_config_listener_v3.Filter {
                StreamIdleTimeout:   envoy.Timeout(b.streamIdleTimeout),
                DrainTimeout:        envoy.Timeout(b.connectionShutdownGracePeriod),
                DelayedCloseTimeout: envoy.Timeout(b.delayedCloseTimeout),
+               StripTrailingHostDot: true,
        }
 
        // Max connection duration is infinite/disabled by default in Envoy, so if the timeout setting

and then run make container to build new container image from the modified code.

@tsaarni we conducted a quick test just now. Prior to the change we sent a request to a test domain with the trailing dot via contour and we saw 404.

Deploying a version of Contour with your suggestion we now see a successful result.

If it could be possible to get a Contour release with a configuration flag for setting StripTrailingHostDot to true it would be a big help for our project. 🙏

@saley89 Glad to hear it works! I'll add "help wanted" label to your issue, but unfortunately I don't have the capacity to work on this feature myself.

@tsaarni no problem. We have attempted a PR for this which is now attached to this ticket.