code

Results 478 comments of code

@derekargueta sorry, I forgot to check the original issue and related comments and different maintainers may have different ideas. IMO, I will still recommend to implement it as part of...

The ci result is weird. Could you merge main and (resolve the conflict) re-push again? Thanks. /wait

Great improvement. But the coverage CI still complains.

Just for curious, basically, the queries are could be treated as part of the HTTP headers. So, I initially think we will do this in the `header_mutation` filter directly. And...

Get it. Yeah, path is special. But path is different with :path. The path sometimes (most times?) would be the part without queries of :path. If we do this under...

See https://github.com/envoyproxy/envoy/pull/34550 for our latest common API for all these map-like mutations. :) To use the new common API, I think you only need only to change the type of...

I have to say sorry to the @derekargueta . Now, this is blocked by the API. We may did incorrect decision in past about the API. And I will try...