search

nucleuscloud / neosync

Open source data anonymization and synthetic data orchestration for developers. Create high fidelity synthetic data and sync it across your environments.
https://www.neosync.dev
Other
3.43k stars 124 forks source link

Adds accountid unary interceptor to backend #2955

Closed nickzelei closed 1 week ago

nickzelei commented 1 week ago

Looks for GetAccountId on the request and adds it to the ctx logger.

Also adds in dedupe of attrs

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **neosync-docs** | ⬜️ Ignored ([Inspect](https://vercel.com/neosync/neosync-docs/C7ndLac7MHbyP5XDcJy1mpcnrNRj)) | [Visit Preview](https://neosync-docs-git-nick-add-account-id-to-log-neosync.vercel.app) | | Nov 19, 2024 0:54am |
codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 55.55556% with 16 lines in your changes missing coverage. Please review.

Project coverage is 35.16%. Comparing base (0e2f00e) to head (176b3b6). Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...rnal/connect/interceptors/accountid/interceptor.go 61.90% 8 Missing :warning:
backend/pkg/logger/logger.go 40.00% 6 Missing :warning:
backend/internal/cmds/mgmt/serve/connect/cmd.go 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2955 +/- ## ========================================== - Coverage 35.17% 35.16% -0.01% ========================================== Files 329 330 +1 Lines 38306 38332 +26 ========================================== + Hits 13473 13480 +7 - Misses 23273 23289 +16 - Partials 1560 1563 +3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features: