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.

Sign up for GitHub

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

Jump to bottom

Use easycaching with elastic apm agent dotnet #461

Open
billhong-just opened this issue Feb 24, 2023 · 4 comments · May be fixed by #510
Open

Use easycaching with elastic apm agent dotnet #461

billhong-just opened this issue Feb 24, 2023 · 4 comments · May be fixed by #510
Assignees
@catcherwong
Labels
feature feature good first issue

Comments

@billhong-just
Copy link
Contributor

Description

Hi, I want to use elastic/apm-agent-dotnet to trace the commands sent to Redis.
Here is a sample code from elastic official document:

// using Elastic.Apm.StackExchange.Redis;

var connection = await ConnectionMultiplexer.ConnectAsync("<redis connection>");
connection.UseElasticApm();

Do I have any chance to get the connection from easycaching or provide a delegate to easycaching?

References
@catcherwong
Copy link
Member

@billhong-just Thanks for your interest in this project.

EasyCaching does not support this one now, I will take a look how to support more APM system.

@catcherwong catcherwong self-assigned this Feb 26, 2023
@billhong-just
Copy link
Contributor Author

@catcherwong Thanks for your quick answer.

Perhaps there could be an event called OnConnected，user can register this event, get the connection and something else when it is established then do what they want.

@catcherwong
Copy link
Member

@billhong-just Could you submit a PR for your idea?

@billhong-just
Copy link
Contributor Author

@catcherwong

Thanks, I will take a look how to make a PR for my idea.
Do you have any contributing guidelines or suggestions? 👋

@sambar-delen sambar-delen linked a pull request Dec 1, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@catcherwong catcherwong

Labels
feature feature good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adding Apm + options to enable it. sambar-delen/EasyCaching
2 participants
@catcherwong @billhong-just