Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: add log field filtering to remove unnecessary metadata #197

Merged
merged 4 commits into from
Nov 14, 2024

Conversation

BravoNatalie
Copy link
Contributor

@BravoNatalie BravoNatalie commented Nov 12, 2024

Description

This PR adds the logDataTransformer callback to the Loki Logging class to filter specific fields in a log.

Motivation

The edge-gateway worker currently produces logs with many unnecessary fields that can be removed to reduce log data size and lower costs. More details can be found in issue #175.

Changes (edge-gateway)

  • Upgraded @web3-storage/worker-utils/loki package to the latest version
  • Upgraded toucan-js to version 3.x for compatibility with @web3-storage/worker-utils/loki
  • Upgraded @sentry/cli to ensure compatibility with toucan-js
  • Added logDataTransformer callback in the Loki Logging constructor for custom log field filtering

@BravoNatalie BravoNatalie self-assigned this Nov 12, 2024
@BravoNatalie BravoNatalie changed the title Refactor: add log field filtering to remove unnecessary metadata refactor: add log field filtering to remove unnecessary metadata Nov 12, 2024
…patibility

- Updated worker-utils package to the latest version
- Updated related Toucan dependencies to ensure compatibility with the newer worker-utils version
@BravoNatalie BravoNatalie marked this pull request as ready for review November 14, 2024 13:40
@BravoNatalie BravoNatalie merged commit 10e9d17 into main Nov 14, 2024
22 checks passed
@BravoNatalie BravoNatalie deleted the refactor/add-log-field-filtering branch November 14, 2024 18:24
BravoNatalie pushed a commit that referenced this pull request Nov 18, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.13.1](edge-gateway-v1.13.0...edge-gateway-v1.13.1)
(2024-11-18)


### Fixes

* trigger release
([74f1e14](74f1e14))
* refactor: add log field filtering to remove unnecessary metadata
[(#197)](#197)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants