Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

fix: Panic when cause is nil while calling client.CaptureError #261

Merged
merged 2 commits into from
Jun 12, 2019

Conversation

misterEggroll
Copy link
Contributor

@misterEggroll misterEggroll commented Jun 11, 2019

  • Return deepest cause which is not nil
  • Test if a nil cause results in a panic

Fixes #260

@kamilogorek kamilogorek merged commit 2111b06 into getsentry:master Jun 12, 2019
@kamilogorek
Copy link
Contributor

Thanks!

@misterEggroll
Copy link
Contributor Author

@kamilogorek Didn't expect it to be merged so fast ! Well done.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Panic if the error implements causer and the cause is nil
2 participants