getsentry / getsentry/sentry-ruby

Patch for async-http

Open
#2,427 1 comment 0 reactions 1 assignee Claimed by @solnic View on GitHub
Feature Ruby Spans
Dominant language
Ruby
Stars
987
Forks
541
Avg merge
17h 40m
Merged PRs (30d)
19

Description

**Describe the idea**
Sentry comes bundled with getting performance spans for Net::Http and a patch for Faraday is easy to enable. Are there any plans for an [async-http](https://github.com/socketry/async-http)? Ruby async is now built into ruby and is becoming much more common place for concurrent IO operations like external API requests.

**Why do you think it's beneficial to most of the users**
When understanding performance issues, external http calls can be a big hit. Currently if using async-http these show up as "Missing Instrumentation" in Sentry. Also, when improving IO performance, moving calls to asynchronous calls is a common tool applied which one would want to see reflected in Sentry perf dashboards.

**Possible implementation**
- A workaround might be using [async-http-faraday](https://github.com/socketry/async-http-faraday)
- It may follow similar lines as https://github.com/getsentry/sentry-ruby/blob/master/sentry-ruby/lib/sentry/faraday.rb

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.