dotnet / dotnet/extensions

HybridCache: implement ReportTagMetrics

Open
#5,989 5 comments 2 reactions 0 assignees View on GitHub
area-caching-hybrid enhancement waiting-on-team :wave:
Dominant language
C#
Stars
3.2k
Forks
894
Avg merge
1d 12h
Merged PRs (30d)
23

Description

### Description

The `HybridCacheOptions` API has a `ReportTagMetrics` member; the intent of this was to prevent leaking PII in any metric reporting; however, currently the metrics *do not include that data*.

We should go back and check what scenarios this may be relevant to, with any / all of:

- adding new tag-specific reporting that is *only* enabled with this enabled
- tweaking existing reporting to include additional data with this enabled

### Reproduction Steps

set `ReportTagMetrics` to... whatever

### Expected behavior

Reporting is richer with this enabled

### Actual behavior

You can toggle it all day long, if you find it cathartic. It doesn't do anything useful.

![useless machine](https://media.tenor.com/EgkVXwM-f9IAAAAM/useless-box-box.gif)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.