Closed tarampampam closed 4 years ago
Merging #43 into master will decrease coverage by
0.81%
. The diff coverage is90.00%
.
@@ Coverage Diff @@
## master #43 +/- ##
============================================
- Coverage 97.70% 96.89% -0.82%
- Complexity 105 117 +12
============================================
Files 23 26 +3
Lines 262 290 +28
============================================
+ Hits 256 281 +25
- Misses 6 9 +3
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
src/Listeners/SendExceptionToStderrListener.php | 0.00% <0.00%> (ø) |
2.00 <2.00> (?) |
|
src/Events/LoopErrorOccurredEvent.php | 100.00% <100.00%> (ø) |
4.00 <4.00> (?) |
|
src/Listeners/StopWorkerListener.php | 100.00% <100.00%> (ø) |
2.00 <2.00> (?) |
|
src/Worker.php | 95.45% <100.00%> (+0.64%) |
26.00 <3.00> (+4.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 0b8c3f5...2f72161. Read the comment docs.
Description
Added
LoopErrorOccurredEvent
(triggered on request processing exception)SendExceptionToStderrListener
for direct exception sending (as a string) intostderr
StopWorkerListener
for worker stoppingChanged
LoopErrorOccurredEvent
event listeners:SendExceptionToStderrListener
andStopWorkerListener
app.debug
) is not enabled - client will get onlyInternal server error
string instead exception with stacktraceFixed
$psr7_client->respond
and$psr7_client->getWorker()->error
after that)Fixes #42
Checklist