Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v3 Branch Pick] Add some error context in transport_failure_callback #1003

Merged
merged 1 commit into from
Aug 28, 2020

Conversation

st0012
Copy link
Collaborator

@st0012 st0012 commented Aug 28, 2020

This picks the changes from #796 with some additional fixes.

Closes #661

* Correct docs re: transactions

* Updated the client failed_send method to include HTTP error when using transport_failure_callback proc

* Updated Readme about transport_failure_callback usage

* Fixed conflicts with v3 previous updates

* Updated Readme
@st0012 st0012 added this to the 3.1.0 milestone Aug 28, 2020
@st0012 st0012 self-assigned this Aug 28, 2020
@codecov-commenter
Copy link

codecov-commenter commented Aug 28, 2020

Codecov Report

Merging #1003 into v3-1 will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             v3-1    #1003   +/-   ##
=======================================
  Coverage   97.85%   97.85%           
=======================================
  Files          46       46           
  Lines        2187     2187           
=======================================
  Hits         2140     2140           
  Misses         47       47           
Impacted Files Coverage Δ
spec/raven/integration_spec.rb 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 c25ea2c...180a8a7. Read the comment docs.

@st0012 st0012 merged commit 43362b8 into v3-1 Aug 28, 2020
@st0012 st0012 deleted the fix-#661 branch August 28, 2020 07:31
st0012 added a commit that referenced this pull request Aug 28, 2020
* Correct docs re: transactions

* Updated the client failed_send method to include HTTP error when using transport_failure_callback proc

* Updated Readme about transport_failure_callback usage

* Fixed conflicts with v3 previous updates

* Updated Readme

Co-authored-by: Etienne Depaulis <[email protected]>
@st0012 st0012 added this to Closed in 3.x Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3.x
  
Closed
Development

Successfully merging this pull request may close these issues.

None yet

3 participants