envoy icon indicating copy to clipboard operation
envoy copied to clipboard

Enrich proxy spans with filter data

Open jcchavezs opened this issue 3 years ago • 6 comments

I was looking at envoy traces generated both when you run a filter and when you don’t and I see traces are pretty much the same except for the fact that when a filter blocks a request, no upstream_cluster/upstream_cluster.name is set (which makes sense). Also there is no clue on the duration of the filter execution (at least in traces) which keeps you blind about the duration of the filter execution. Finally, AFAIU filters have no way to access the current span to set attributes or annotations (e.g. modsecurity won’t tell you what rule caused the blocking).

In summary what I’d like to see as outcome would be:

  • traces display filter duration either as annotations in the trace (subspans might be overkilling I think) filters can enrich span attributes
  • Have an attribute about which filter blocked the request.

In the future I'd like it to be included in the WASM ABI too.

jcchavezs avatar Jun 30 '22 10:06 jcchavezs

traces display filter duration either as annotations in the trace (subspans might be overkilling I think) filters can enrich span attributes

Sound great. But it would bring some additional overhead. 🤔

Have an attribute about which filter blocked the request.

May be response flag or response detail string is better for this requirement?

wbpcode avatar Jul 05 '22 01:07 wbpcode

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

github-actions[bot] avatar Aug 04 '22 04:08 github-actions[bot]

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted" or "no stalebot". Thank you for your contributions.

github-actions[bot] avatar Aug 11 '22 08:08 github-actions[bot]

This issue has been closed by a bot with zero context about the status nor use case.

jcchavezs avatar Aug 19 '22 11:08 jcchavezs

it was closed due to inactivity - i can reopen it now - but you will need to either implement this yourself, or find someone willing to do so, as well as get it approved

phlax avatar Aug 19 '22 13:08 phlax

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

github-actions[bot] avatar Sep 18 '22 16:09 github-actions[bot]

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted" or "no stalebot". Thank you for your contributions.

github-actions[bot] avatar Sep 25 '22 20:09 github-actions[bot]