Skip to content

Conversation

@sunjayBhatia
Copy link
Member

For a Gateway HTTPS Listeners with unspecified (catchall/wildcard) hostnames, we add a wildcard filter chain

This is to ensure we get a routing failure HTTP response (404) when a request is received that is for an "unknown" fqdn, rather than a connection error because Envoy does not have a filter chain to match

For a Gateway HTTPS Listeners with unspecified (catchall/wildcard) hostnames, we add a wildcard filter chain

This is to ensure we get a routing failure HTTP response (404) when a request is received that is for an "unknown" fqdn, rather than a connection error because Envoy does not have a filter chain to match

Signed-off-by: Sunjay Bhatia <[email protected]>
@sunjayBhatia sunjayBhatia requested a review from a team as a code owner November 17, 2025 20:08
@sunjayBhatia sunjayBhatia added the release-note/small A small change that needs one line of explanation in the release notes. label Nov 17, 2025
@sunjayBhatia sunjayBhatia requested review from a team, clayton-gonsalves, rajatvig and tsaarni and removed request for a team November 17, 2025 20:08
Signed-off-by: Sunjay Bhatia <[email protected]>
@codecov
Copy link

codecov bot commented Nov 17, 2025

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.13%. Comparing base (30397f1) to head (7d7c19c).
⚠️ Report is 32 commits behind head on main.

Files with missing lines Patch % Lines
internal/dag/gatewayapi_processor.go 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7306      +/-   ##
==========================================
- Coverage   81.85%   80.13%   -1.72%     
==========================================
  Files         130      130              
  Lines       15747    15785      +38     
==========================================
- Hits        12889    12649     -240     
- Misses       2574     2609      +35     
- Partials      284      527     +243     
Files with missing lines Coverage Δ
internal/dag/dag.go 87.59% <100.00%> (-10.86%) ⬇️
internal/dag/gatewayapi_processor.go 75.74% <66.66%> (-18.14%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

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

This bot triages PRs according to the following rules:

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

You can:

  • Ensure your PR is passing all CI checks. PRs that are fully green are more likely to be reviewed. If you are having trouble with CI checks, reach out to the #contour channel in the Kubernetes Slack workspace.
  • Mark this PR as fresh by commenting or pushing a commit
  • Close this PR
  • Offer to help out with triage

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

@github-actions github-actions bot added lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/small A small change that needs one line of explanation in the release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants