roadrunner-server / roadrunner-plugins

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

feat(http, new_relic): handle new_relic error from the PHP worker #94

Closed rustatian closed 2 years ago

rustatian commented 2 years ago

Reason for This PR

Description of Changes

New Relic errors from the PHP are handled by the special HTTP header - rr_newrelic_error. Predefined values:

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 2 years ago

Codecov Report

Merging #94 (9320a95) into master (2cfc2da) will decrease coverage by 0.81%. The diff coverage is 52.94%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
- Coverage   68.57%   67.75%   -0.82%     
==========================================
  Files         118      123       +5     
  Lines        9947    10099     +152     
==========================================
+ Hits         6821     6843      +22     
- Misses       2454     2582     +128     
- Partials      672      674       +2     
Impacted Files Coverage Δ
http/middleware/new_relic/plugin.go 0.00% <0.00%> (ø)
http/middleware/new_relic/splitter.go 80.00% <80.00%> (ø)
http/middleware/new_relic/error.go 100.00% <100.00%> (ø)
boltdb/boltjobs/listener.go 58.49% <0.00%> (-2.84%) :arrow_down:
sqs/sqsjobs/item.go 65.78% <0.00%> (-1.32%) :arrow_down:
http/middleware/new_relic/writer.go 0.00% <0.00%> (ø)
http/middleware/new_relic/config.go 0.00% <0.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 2cfc2da...9320a95. Read the comment docs.