@ryandens it seems that snyk still fails on the PRs from forks.

The CI passes not. Snyk is misconfigured and fails on PRs from forks on

"error-message": "snyk requires an authenticated account. Please run snyk auth and try again.",

The body capture works, tested on https://github.com/grpc/grpc-java/tree/master/examples

do not merge yet it needs more love - e.g. remove the filter paths from the instrumentation config. That should not be available to instrumentations.

@shashank11p PR updated, could you please re-review?

@ryandens it seems that snyk still fails on the PRs from forks.

Hey Pavol! Sorry about that would you mind trying out a fix to help this PR along? I think we need to

1. Swap out --org=hypertrace with --org=$GITHUB_ACTOR
2. Depending on whether you've done this before, you'll need to login and authorize your github account.

I am not sure if that will work, most likely the secret is not present in PRs from forks https://github.com/pavolloffay/javaagent/blob/52f0a2d388ce5951e4627f6f18c0fa0f9e4f12a1/.github/workflows/build.yaml#L82.

This repository has the same snyk setting as other HT repos e.g. https://github.com/hypertrace/hypertrace-service/blob/main/.github/workflows/pr-build.yml#L79. I will leave this as it is, somebody from the team should fix it so that PRs from the community are not turning red.

EDIT: snyk passed on hypertrace/hypertrace-service#97. I will look at it in a different PR