cloudfoundry-attic / cf-abacus

CF usage metering and aggregation
Apache License 2.0
98 stars 86 forks source link

Validate sampler receiver requests #1065

Closed martin-d-aleksandrov closed 6 years ago

cfdreddbot commented 6 years ago

Hey martin-d-aleksandrov!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

cf-gitbot commented 6 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/161014638

The labels on this github issue will be updated when the story is started.

codecov[bot] commented 6 years ago

Codecov Report

Merging #1065 into master will increase coverage by 0.04%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1065      +/-   ##
==========================================
+ Coverage   95.02%   95.07%   +0.04%     
==========================================
  Files         117      119       +2     
  Lines        3375     3409      +34     
==========================================
+ Hits         3207     3241      +34     
  Misses        168      168
Impacted Files Coverage Δ
lib/sampler/receiver/src/lib/event-validator.js 100% <100%> (ø)
...pler/receiver/src/lib/json-media-type-validator.js 100% <100%> (ø)
lib/utils/api/src/lib/receiver-client.js 100% <100%> (ø) :arrow_up:
lib/utils/api/src/lib/errors.js 100% <100%> (ø) :arrow_up:
lib/sampler/receiver/src/lib/presenter.js 100% <100%> (ø) :arrow_up:

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 9689f75...f982a1e. Read the comment docs.