Michael Warres
Michael Warres
> Hi, @mpwarres @kyessenov , specific to wasm module, I am basically a beginner. Could you take a quick look at this to see if this make sense or in...
ISTM we'd need to add a new `proxy_metric_type_t` enum value, and a new variation of `proxy_record_metric` that accepts a string. @PiotrSikora WDYT?
@kyessenov FYI
@leonm1 @martijneken FYI
@PiotrSikora one item not in the current list is #54 , for async KV get/set. Does that fall into the "Everything else can be done either using feature flags or...
> @mpwarres could we get someone from Google to convert Rust tests in Envoy to use the `header-value` feature (it's already exposed via `@proxy_wasm_rust_sdk//:proxy_wasm_header_value` Bazel target), to verify that I...
I put together a doc that consolidates information about this issue and evaluates options: [Envoy Wasm / Proxy-Wasm support for FilterHeadersStatus::StopIteration](https://docs.google.com/document/d/1Whd1C0k-H2NHrPOmlAqqauFz6ObSTP017juJIYyciB0/edit?usp=sharing) tl;dr is that I think we should define a...
> I don't recall why this is implemented the way it is, but there are valid use cases where you want callouts to outlive the lifetime of the primary connection...
``` ERROR: /home/runner/.cache/bazel/_bazel_runner/ae21fb6b2e26a9267f16b46bb90ea321/external/v8/bazel/defs.bzl:472:22: Use of Starlark transition without allowlist attribute '_allowlist_function_transition'. See Starlark transitions documentation for details and usage: @v8//:bazel/defs.bzl NORMAL ``` Looks like I need to add a patch...
> They removed it quite recently (in V8 v12.4, see: [v8/v8@b26554e](https://github.com/v8/v8/commit/b26554ec368e9553782012c96aa5e99b163eaff2)), so you should be able to simply revert that commit. Thanks for the pointer; updated patch to revert that...