intuit / QuickBooks-V3-DotNET-SDK

.Net SDK for QuickBooks REST API v3 services
Apache License 2.0
107 stars 140 forks source link

Allow providing a custom HttpMessageHandler #257

Open czietsman opened 2 years ago

czietsman commented 2 years ago

I would like to be able specify a custom HTTP message handler so that I can capture the 'intuit_tid' header and add it to our logs.

I've added an optional HttpMessageHandler innerHandler = null where it made sense and passed it through.

The following decisions were made:

nimisha84 commented 1 year ago

Thanks for the PR. I had moved to a different team and SDK ownership has been transferred. CLosing off the pending PRs. Looks like with this change, if existing devs directly call the different oauth lib clients, they can break. I would suggest creating another constructor instead of updating the existing one for your use case. Please update us when you are ready with the PR.