envoy icon indicating copy to clipboard operation
envoy copied to clipboard

uhv: disable oghttp2 header validation when uhv is enabled

Open ameily opened this issue 3 years ago • 9 comments

Commit Message: uhv: disable oghttp2 header validation when uhv is enabled Additional Description: Disable header validation within oghttp2 when the UHV compiler flag is enabled Risk Level: Low Testing: Docs Changes: Release Notes: Platform Specific Features: Fixes: #21548 Fixes: #19751

ameily avatar Jul 08 '22 19:07 ameily

CC @yanavlasov

ameily avatar Jul 09 '22 13:07 ameily

/assign @yanavlasov

adisuissa avatar Jul 11 '22 18:07 adisuissa

/retest

phlax avatar Jul 20 '22 11:07 phlax

Retrying Azure Pipelines: Retried failed jobs in: envoy-presubmit

:cat:

Caused by: a https://github.com/envoyproxy/envoy/pull/22078#issuecomment-1190142342 was created by @phlax.

see: more, trace.

@ameily i think this probably needs a main merge - lets see...

yep - it does

/wait

phlax avatar Jul 20 '22 11:07 phlax

/wait

phlax avatar Jul 20 '22 11:07 phlax

@phlax I rebased on main. CI is failing a couple steps that I think just need to be run again.

  • linux coverage: //test/integration:tcp_tunneling_integration_test fails, but passes locally for me
  • windows: failed to download bazel

ameily avatar Jul 21 '22 16:07 ameily

yep, looks like flakes - ill follow up on the first one - i think ive seen it elsewhere

/retest

phlax avatar Jul 21 '22 16:07 phlax

Retrying Azure Pipelines: Retried failed jobs in: envoy-presubmit

:cat:

Caused by: a https://github.com/envoyproxy/envoy/pull/22078#issuecomment-1191700923 was created by @phlax.

see: more, trace.