search

Connehito / cake-sentry

CakePHP plugin integration for Sentry
MIT License
34 stars 26 forks source link

Make CakePHP Logger context data to be recorded as extra data #68

Closed o0h closed 3 years ago

o0h commented 3 years ago

On calling Log::log() directly, CakePHP allows you to pass a context argument. In cake-sentry, this information should not be ignored but should be treated as such.

In this PR, it will be sent as extra data.

codecov-commenter commented 3 years ago

Codecov Report

Merging #68 (a234f00) into 4.x (56612f4) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##                 4.x       #68   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        18        18           
===========================================
  Files              4         4           
  Lines             62        65    +3     
===========================================
+ Hits              62        65    +3
Impacted Files Coverage Δ
src/Http/Client.php 100.00% <100.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 56612f4...a234f00. Read the comment docs.