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

Add support to publish custom analytics data from APIM #11349

Merged
merged 2 commits into from
Jul 29, 2022

Conversation

YasasRangika
Copy link
Contributor

Purpose

To add support to publish custom analytics data from APIM by including any attributes to the existing event schema.

Goal

Fixes: wso2/api-manager#443

Approach

@codecov
Copy link

codecov bot commented Jul 27, 2022

Codecov Report

Merging #11349 (f779e0a) into master (b73dbf1) will decrease coverage by 2.23%.
The diff coverage is 7.93%.

@@             Coverage Diff              @@
##             master   #11349      +/-   ##
============================================
- Coverage     47.62%   45.38%   -2.24%     
+ Complexity     5186     4757     -429     
============================================
  Files          1740     1742       +2     
  Lines        124422   124620     +198     
  Branches      17046    17079      +33     
============================================
- Hits          59257    56562    -2695     
- Misses        58331    61300    +2969     
+ Partials       6834     6758      -76     
Flag Coverage Δ
integration_tests 39.23% <7.93%> (-3.10%) ⬇️
unit_tests 20.39% <0.00%> (-0.04%) ⬇️

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

Impacted Files Coverage Δ
...on/analytics/collectors/AnalyticsDataProvider.java 0.00% <0.00%> (ø)
...cs/collectors/impl/FaultyRequestDataCollector.java 0.00% <0.00%> (ø)
...s/collectors/impl/SuccessRequestDataCollector.java 0.00% <0.00%> (ø)
.../apimgt/common/analytics/publishers/dto/Event.java 0.00% <0.00%> (ø)
...ay/handlers/analytics/AnalyticsMetricsHandler.java 0.00% <0.00%> (ø)
...n/apimgt/gateway/handlers/analytics/Constants.java 0.00% <ø> (ø)
...ndlers/analytics/SynapseAnalyticsDataProvider.java 0.00% <0.00%> (ø)
...ming/websocket/WebSocketAnalyticsDataProvider.java 0.00% <0.00%> (ø)
...ng/websocket/WebSocketAnalyticsMetricsHandler.java 0.00% <0.00%> (ø)
...pimgt/gateway/internal/ServiceReferenceHolder.java 71.53% <26.31%> (-7.23%) ⬇️
... and 160 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b73dbf1...f779e0a. Read the comment docs.

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.

Add support to publish custom attributes to ELK based Analytics
3 participants