recurly / recurly-client-dotnet

A .NET API wrapper for Recurly.
https://developers.recurly.com
MIT License
86 stars 80 forks source link

feat add event handlers to make request async #811

Closed aron-ridgway-deltatre closed 10 months ago

aron-ridgway-deltatre commented 10 months ago

Issue we are facing

The issue we are facing is we have a multi tenant application and we need to add tracing/monitoring for Recurly http calls per tenant, so we can monitor the amount off calls being made by each tenant which include paged requests that happen in the background. BaseClient MakeRequest has Event handlers which can be used to hook into http request and responses. MakeRequestAsync that we use in our service is currently missing the Event Handling processing,

Proposed solution

Add the event handling logic to the MakeRequestAsync method.

There may be a better way to solve this issue, so we are open to your suggestions on the best implementation to resolve this issue.