roadrunner-server / roadrunner-plugins

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

panic(tls): panic in the tls layer when the user does not use http #96

Closed rustatian closed 3 years ago

rustatian commented 3 years ago

Reason for This PR

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 #96 (3de11a2) into master (7179229) will increase coverage by 0.16%. The diff coverage is 76.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   67.70%   67.87%   +0.16%     
==========================================
  Files         123      123              
  Lines       10099    10093       -6     
==========================================
+ Hits         6838     6851      +13     
+ Misses       2586     2565      -21     
- Partials      675      677       +2     
Impacted Files Coverage Δ
http/config/tls.go 52.17% <ø> (ø)
http/http.go 52.94% <ø> (+7.94%) :arrow_up:
http/ssl.go 30.30% <ø> (-5.81%) :arrow_down:
http/serve.go 71.03% <25.00%> (+3.44%) :arrow_up:
http/plugin.go 78.73% <100.00%> (+0.66%) :arrow_up:
sqs/sqsjobs/item.go 67.10% <0.00%> (+1.31%) :arrow_up:
redis/pubsub/pubsub.go 75.19% <0.00%> (+2.32%) :arrow_up:
... and 1 more

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 7179229...3de11a2. Read the comment docs.