tyxia

Results 155 comments of tyxia

/retest The CI failure should has nothing to do with this PR

> @tyxia i think the tsan problem has been fixed on `main` here #24252 > > the other problem is coverage, which is saying: > > ``` > Code coverage...

/assign @yanavlasov

@dastrobu For your example, do you see `foo-bar: value1, value2` ? That is expected, ext_authz [merge duplicate header with `, `](https://github.com/envoyproxy/envoy/blob/f301eebf7acc680e27e03396a1be6be77e1ae3a5/source/extensions/filters/common/ext_authz/check_request_utils.cc#L150-L151) . This behavior strictly follow the RFC https://www.rfc-editor.org/rfc/rfc7230#section-3.2 Merging...