Closed MrCreosote closed 5 years ago
Merging #112 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #112 +/- ##
=======================================
Coverage 91.38% 91.38%
=======================================
Files 14 14
Lines 1543 1543
=======================================
Hits 1410 1410
Misses 99 99
Partials 34 34
Impacted Files | Coverage Δ | |
---|---|---|
service/server.go | 97.81% <ø> (ø) |
: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 8fdd744...957465c. Read the comment docs.
Merging #112 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #112 +/- ##
=======================================
Coverage 91.38% 91.38%
=======================================
Files 14 14
Lines 1543 1543
=======================================
Hits 1410 1410
Misses 99 99
Partials 34 34
Impacted Files | Coverage Δ | |
---|---|---|
service/server.go | 97.81% <ø> (ø) |
: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 8fdd744...957465c. Read the comment docs.
Also remove some TODOs:
Default max header len is 1MB, so adding a timeout as well doesn't really seem necessary.
For details: https://blog.cloudflare.com/the-complete-guide-to-golang-net-http-timeouts/