Skip to content
This repository has been archived by the owner on Sep 2, 2022. It is now read-only.

Send request and response body size information with metrics #14

Merged
merged 4 commits into from Feb 1, 2022

Conversation

ruohola
Copy link
Member

@ruohola ruohola commented Feb 1, 2022

Just have to rely on Content-Length headers here since there really didn't seem to be any other reliable way of getting request or response body sizes.

@ruohola ruohola changed the title Send request and response body size information with metrics Send request and response body size information with metrics Feb 1, 2022
@ruohola ruohola requested a review from blomqma February 1, 2022 22:23
Also, replace the clunky EndBug/add-and-commit action simply with good
old CLI Git usage.

Was nice to get rid of clean-git-head.js as well.
@codecov-commenter
Copy link

codecov-commenter commented Feb 1, 2022

Codecov Report

Merging #14 (fba9d5c) into master (71cec49) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           59        68    +9     
  Branches        10        16    +6     
=========================================
+ Hits            59        68    +9     
Impacted Files Coverage Δ
packages/core/src/index.ts 100.00% <ø> (ø)
packages/express/src/index.ts 100.00% <100.00%> (ø)
packages/next/src/index.ts 100.00% <100.00%> (ø)

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 71cec49...fba9d5c. Read the comment docs.

Just have to rely on Content-Length headers here since there really
didn't seem to be any other reliable way of getting request or response
body sizes.
@ruohola ruohola merged commit e1d819c into master Feb 1, 2022
@ruohola ruohola deleted the request-response-sizes branch February 1, 2022 22:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants