Closed GiedriusM closed 6 years ago
Merging #20 into master-rest will increase coverage by
0.7%
. The diff coverage is70.37%
.
@@ Coverage Diff @@
## master-rest #20 +/- ##
==============================================
+ Coverage 65.04% 65.74% +0.7%
==============================================
Files 9 9
Lines 615 648 +33
==============================================
+ Hits 400 426 +26
- Misses 215 222 +7
Impacted Files | Coverage Δ | |
---|---|---|
examples/rest-server/rest-core.c | 53.12% <100%> (+8.68%) |
:arrow_up: |
examples/rest-server/rest-endpoints.c | 87.5% <100%> (+0.7%) |
:arrow_up: |
examples/rest-server/restserver.c | 73.68% <100%> (+0.95%) |
:arrow_up: |
examples/rest-server/rest-notifications.c | 55.2% <27.27%> (-0.64%) |
:arrow_down: |
examples/rest-server/rest-resources.c | 48.59% <64.28%> (+2.13%) |
:arrow_up: |
examples/rest-server/rest-subscriptions.c | 76.47% <75%> (-0.2%) |
:arrow_down: |
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 508cccd...a1bd4c3. Read the comment docs.
Ulfius calls each request callback from a new thread. This PR adds mutex to rest context that should prevent data corruption and the arising segfaults.