roadrunner-server / roadrunner-plugins

📦 Home for the roadrunner plugins
MIT License
25 stars 9 forks source link

fix(grpc): grpc plugin fails to handle requests after reset #70

Closed rustatian closed 3 years ago

rustatian commented 3 years ago

Reason for This PR

closes: #69

Description of Changes

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.] [Reviewer TODO: Verify that these criteria are met. Request changes if not]

codecov[bot] commented 3 years ago

Codecov Report

Merging #70 (332bb2c) into master (7e65ed5) will decrease coverage by 0.11%. The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   68.53%   68.41%   -0.12%     
==========================================
  Files         112      112              
  Lines        9314     9327      +13     
==========================================
- Hits         6383     6381       -2     
- Misses       2302     2318      +16     
+ Partials      629      628       -1     
Impacted Files Coverage Δ
jobs/protocol.go 48.83% <37.50%> (-3.80%) :arrow_down:
grpc/plugin.go 62.01% <100.00%> (+15.61%) :arrow_up:
grpc/proxy/proxy.go 62.67% <100.00%> (+0.53%) :arrow_up:
grpc/server.go 65.76% <100.00%> (+0.31%) :arrow_up:
jobs/metrics.go 95.91% <100.00%> (-4.09%) :arrow_down:
jobs/listener.go 38.59% <0.00%> (-14.04%) :arrow_down:
sqs/item.go 64.78% <0.00%> (-6.34%) :arrow_down:
jobs/plugin.go 63.27% <0.00%> (-0.50%) :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 7e65ed5...332bb2c. Read the comment docs.