Skip to content

fix: Silently catch exception from offer() if channel is closed.#86

Merged
arriolac merged 1 commit into
mainfrom
chris/fix/85
Aug 27, 2020
Merged

fix: Silently catch exception from offer() if channel is closed.#86
arriolac merged 1 commit into
mainfrom
chris/fix/85

Conversation

@arriolac

Copy link
Copy Markdown
Contributor

Fix scenario where app can crash due to unhandled exception from invoking offer() on a closed channel.

Fixes #85

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 27, 2020
@codecov

codecov Bot commented Aug 27, 2020

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (main@d1df1a9). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##             main     #86   +/-   ##
======================================
  Coverage        ?   0.00%           
======================================
  Files           ?      28           
  Lines           ?     191           
  Branches        ?      27           
======================================
  Hits            ?       0           
  Misses          ?     191           
  Partials        ?       0           

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 d1df1a9...654eb03. Read the comment docs.

@barbeau barbeau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arriolac arriolac changed the title fix: Silently catch exception if channel is closed. fix: Silently catch exception from offer() if channel is closed. Aug 27, 2020
@arriolac arriolac merged commit 8a8fc7c into main Aug 27, 2020
@arriolac arriolac deleted the chris/fix/85 branch August 27, 2020 22:08
googlemaps-bot pushed a commit that referenced this pull request Aug 27, 2020
## [2.1.2](v2.1.1...v2.1.2) (2020-08-27)

### Bug Fixes

* Silently catch exception if channel is closed. ([#86](#86)) ([8a8fc7c](8a8fc7c))
@googlemaps-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.1.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Offer can throw, crashing the app on edge cases where cancellation happens in another thread

4 participants