Skip to content
This repository has been archived by the owner on Jun 13, 2023. It is now read-only.

feat(index.js): support ignoring payloads for Python (#106) #107

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

ojongerius
Copy link
Contributor

Fixes #106 but is blocked on epsagon/epsagon-python#373

Example usage:

  epsagon:
    token: "foo"
    appName: "bar"
    payloadsToIgnore:
      - foo: bar
      - baz: quuz

Which results in generated wrapper code getting this line added, which in turn can is used by the python library:

<SNIP>
os.environ['EPSAGON_PAYLOADS_TO_IGNORE'] = '[{"foo":"bar"},{"baz":"quuz"}]' if 'EPSAGON_PAYLOADS_TO_IGNORE' not in os.environ else os.environ['EPSAGON_PAYLOADS_TO_IGNORE']
<SNIP>

ojongerius added a commit to epsagon/epsagon-python that referenced this pull request Aug 4, 2021
Alon-Katz pushed a commit to epsagon/epsagon-python that referenced this pull request Aug 4, 2021
…armup plugin (#373)

* feat(aws_lambda): do not trace warmup events from serverless-plugin-warmup plugin

* feat(aws_lambda): do not trace list of payloads to ignore, configured using environment variable

* fix(aws_lambda): Expect proper array / list of dictionaries instead

Works a little nicer with epsagon/serverless-plugin-epsagon#107
@ojongerius
Copy link
Contributor Author

@ranrib / @Alon-Katz time to review?

@ojongerius
Copy link
Contributor Author

If raised epsagon/epsagon-node#508 to add node support, if that goes in before this one the README should be updated that payloadsToIgnore is available for Node too.

@ranrib
Copy link
Member

ranrib commented Aug 5, 2021

I sent you an invite to this repo as well @ojongerius 🙂

@ojongerius
Copy link
Contributor Author

New PR at epsagon/epsagon-node#509

Copy link
Member

@ranrib ranrib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@ranrib ranrib merged commit 2f13dd0 into epsagon:master Aug 5, 2021
@ranrib
Copy link
Member

ranrib commented Aug 5, 2021

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ranrib ranrib added the released label Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow ignoring tracing of configured payloads
2 participants