Skip to content

Adding support for stream_error_on_invalid_http_message #7126

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Adding support for stream_error_on_invalid_http_message #7126

wants to merge 3 commits into from

Conversation

YashNandwana
Copy link

Fixes #5598

@YashNandwana YashNandwana requested a review from a team as a code owner July 11, 2025 22:24
@YashNandwana YashNandwana requested review from tsaarni and sunjayBhatia and removed request for a team July 11, 2025 22:24
@sunjayBhatia sunjayBhatia requested review from a team, wilsonwu and davinci26 and removed request for a team July 11, 2025 22:24
@YashNandwana YashNandwana changed the title Addin support for stream_error_on_invalid_http_message Adding support for stream_error_on_invalid_http_message Jul 11, 2025
@github-actions GitHub Actions
Copy link

Hi @YashNandwana! Welcome to our community and thank you for opening your first Pull Request. Someone will review it soon. Thank you for committing to making Contour better. You can also join us on our mailing list and in our channel in the Kubernetes Slack Workspace

@YashNandwana YashNandwana mentioned this pull request Jul 11, 2025
Open
@YashNandwana YashNandwana force-pushed the feature/downstream-validation branch from 5c637d9 to c43da05 Compare July 13, 2025 17:34
@YashNandwana YashNandwana marked this pull request as draft July 13, 2025 17:35
@YashNandwana YashNandwana force-pushed the feature/downstream-validation branch from c43da05 to 5c637d9 Compare July 13, 2025 17:39
@YashNandwana YashNandwana marked this pull request as ready for review July 13, 2025 18:01
@sunjayBhatia sunjayBhatia requested a review from a team July 13, 2025 18:01
@YashNandwana YashNandwana marked this pull request as draft July 13, 2025 18:02
@YashNandwana YashNandwana force-pushed the feature/downstream-validation branch from c9d57e1 to e49a07f Compare July 13, 2025 18:05
@YashNandwana
adding downstream validation for envoy
a691327 
Signed-off-by: YashNandwana <yashneelam975@gmail.com>
@YashNandwana
adding downstream validation for envoy (streamErrorOnInvalidHttpMessage)
2cdcd25 
Signed-off-by: YashNandwana <yashneelam975@gmail.com>
@YashNandwana
added to OnChange() for getting loaded during DAG build
bb28957 
Signed-off-by: YashNandwana <yashneelam975@gmail.com>
@YashNandwana YashNandwana force-pushed the feature/downstream-validation branch from e49a07f to bb28957 Compare July 13, 2025 18:07
@YashNandwana YashNandwana marked this pull request as ready for review July 13, 2025 18:12
@github-actions GitHub Actions
Copy link

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 the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tsaarni tsaarni Awaiting requested review from tsaarni tsaarni is a code owner automatically assigned from projectcontour/maintainers

@sunjayBhatia sunjayBhatia Awaiting requested review from sunjayBhatia sunjayBhatia is a code owner automatically assigned from projectcontour/maintainers

@wilsonwu wilsonwu Awaiting requested review from wilsonwu wilsonwu was automatically assigned from projectcontour/contour-reviewers

@davinci26 davinci26 Awaiting requested review from davinci26 davinci26 was automatically assigned from projectcontour/contour-reviewers

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

How to setup contour as level 2 proxy
1 participant
@YashNandwana