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

Improve: Analytics Event Data Masking #12190

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

BLasan
Copy link
Contributor

@BLasan BLasan commented Nov 29, 2023

@BLasan BLasan force-pushed the analytics-data-masking branch 2 times, most recently from 7510820 to 1202f23 Compare November 29, 2023 08:04
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

Attention: Patch coverage is 36.36364% with 42 lines in your changes are missing coverage. Please review.

Project coverage is 47.26%. Comparing base (bb82dad) to head (7b78b4e).
Report is 792 commits behind head on master.

Files Patch % Lines
...s/collectors/impl/SuccessRequestDataCollector.java 14.70% 23 Missing and 6 partials ⚠️
...o2/carbon/apimgt/impl/APIManagerConfiguration.java 57.14% 6 Missing and 3 partials ⚠️
...ming/websocket/WebSocketAnalyticsDataProvider.java 0.00% 3 Missing ⚠️
...ndlers/analytics/SynapseAnalyticsDataProvider.java 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #12190      +/-   ##
============================================
+ Coverage     39.14%   47.26%   +8.11%     
- Complexity        0     4118    +4118     
============================================
  Files          1785     1840      +55     
  Lines        130478   136523    +6045     
  Branches      18803    19763     +960     
============================================
+ Hits          51081    64526   +13445     
+ Misses        72902    64386    -8516     
- Partials       6495     7611    +1116     
Flag Coverage Δ
integration_tests 41.99% <34.84%> (+10.53%) ⬆️
unit_tests 20.42% <15.62%> (-0.41%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BLasan BLasan marked this pull request as draft January 8, 2024 05:13
@BLasan BLasan force-pushed the analytics-data-masking branch from c515800 to 7b78b4e Compare March 21, 2024 04:19
@BLasan BLasan marked this pull request as ready for review March 21, 2024 04:22
@tharikaGitHub tharikaGitHub merged commit dcc872c into wso2:master Apr 2, 2024
8 of 9 checks passed
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.

Masking User IP value
3 participants