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

Can we get instrumentation support for Net/SMTP? #1294

Open
kohder opened this issue Dec 22, 2020 · 6 comments
Open

Can we get instrumentation support for Net/SMTP? #1294

kohder opened this issue Dec 22, 2020 · 6 comments
Labels
community Was opened by a community member feature-request A request for a new feature or change to an existing one integrations Involves tracing integrations
Projects

Comments

@kohder
Copy link

kohder commented Dec 22, 2020

We have an internal system that sends a lot of email via Ruby's Net/SMTP. We'd like to be able to instrument these SMTP calls. Looks like it would be similar to the existing integration for Net/HTTP.
Thanks!

@marcotc
Copy link
Member

marcotc commented Dec 23, 2020

Hey @kohder, this seems like a straightforward suggestion. I'll add it to our feature request list.
I doesn't seem too different from our existing Net::HTTP integration, so I don't see many complications on this one.

@marcotc marcotc added community Was opened by a community member feature-request A request for a new feature or change to an existing one integrations Involves tracing integrations labels Dec 23, 2020
@delner delner added this to Needs triage in Planning via automation Apr 15, 2021
@mobilutz
Copy link

I see that this integration is in Planning since April.

Is there some update on when this is in development?

Looking forward to this.

@ivoanjo
Copy link
Member

ivoanjo commented Sep 6, 2021

Hey y'all! Sorry for the radio silence; we're still considering this feature but haven't picked it up yet.

I'll unassign @marcotc to make it clear that it's yet to be decided.

Please do comment if this is important for you, as it does get push things closer to the top of the list 😄

@marcotc
Copy link
Member

marcotc commented Sep 10, 2021

We have a related update: ActionMailer integration has been merged and will be shipped with the next ddtrace release.

If you happen to use Net::SMTP through ActionMailer this integration might provide the visibility needed.

I understand that this does not address all Net::SMTP use cases, but I think it's worth bringing up in this thread.

@paneq
Copy link
Contributor

paneq commented Sep 5, 2023

@marcotc I think you meant Net::SMTP in the previous message.

@marcotc
Copy link
Member

marcotc commented Sep 5, 2023

@paneq, yes, edited, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Was opened by a community member feature-request A request for a new feature or change to an existing one integrations Involves tracing integrations
Projects
Planning
Needs triage
Development

No branches or pull requests

5 participants