Skip to content

feat: HTTPRoute support 303 307 308 redirect#8182

Merged
zirain merged 4 commits intoenvoyproxy:mainfrom
zirain:httproute-redirect
Mar 2, 2026
Merged

feat: HTTPRoute support 303 307 308 redirect#8182
zirain merged 4 commits intoenvoyproxy:mainfrom
zirain:httproute-redirect

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented Feb 4, 2026

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 4, 2026

Deploy Preview for cerulean-figolla-1f9435 ready!

Name Link
🔨 Latest commit c974d99
🔍 Latest deploy log https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/699d62743761720008c24ca7
😎 Deploy Preview https://deploy-preview-8182--cerulean-figolla-1f9435.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@zirain zirain force-pushed the httproute-redirect branch from fd052e5 to 59d7da8 Compare February 5, 2026 13:29
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.74%. Comparing base (d1b1e5c) to head (c974d99).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8182      +/-   ##
==========================================
+ Coverage   73.72%   73.74%   +0.02%     
==========================================
  Files         240      241       +1     
  Lines       36917    37049     +132     
==========================================
+ Hits        27216    27321     +105     
- Misses       7772     7787      +15     
- Partials     1929     1941      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@zirain zirain marked this pull request as ready for review February 5, 2026 23:52
@zirain zirain requested a review from a team as a code owner February 5, 2026 23:52
@zirain zirain force-pushed the httproute-redirect branch from a8940f6 to c5ab12c Compare February 6, 2026 10:39
@arkodg
Copy link
Copy Markdown
Contributor

arkodg commented Feb 8, 2026

prefer waiting until we actually bump gateway api version to be safe that this actually works end to end

@arkodg arkodg added this to the v1.8.0-rc.1 Release milestone Feb 8, 2026
kkk777-7
kkk777-7 previously approved these changes Feb 12, 2026
@kkk777-7
Copy link
Copy Markdown
Member

LGTM! (wait until bump gwapi version)

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Feb 13, 2026

v1.5 will be released in two weeks, let's wait for it.

Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
@zirain
Copy link
Copy Markdown
Member Author

zirain commented Feb 26, 2026

@arkodg @kkk777-7 this's ready for review.

@arkodg arkodg requested review from a team March 2, 2026 02:26
@kkk777-7
Copy link
Copy Markdown
Member

kkk777-7 commented Mar 2, 2026

LGTM, thanks!

@zirain zirain merged commit e45e248 into envoyproxy:main Mar 2, 2026
114 of 128 checks passed
@zirain zirain deleted the httproute-redirect branch March 2, 2026 10:42
antonio-mazzini pushed a commit to antonio-mazzini/gateway that referenced this pull request Mar 5, 2026
* feat: HTTPRoute support 303 307 308 redirect

Signed-off-by: zirain <zirain2009@gmail.com>

* add test

Signed-off-by: zirain <zirain2009@gmail.com>

* fix lint

Signed-off-by: zirain <zirain2009@gmail.com>

* remove skipped tests

Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: zirain <zirain2009@gmail.com>
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