EasyPost / easypost-csharp

EasyPost Shipping API Client Library for .NET and .NET Framework applications

https://easypost.com/docs/api

MIT License

66 stars 74 forks source link

[fix] Recursion causing stack overflow when disposing client/service #588

Closed nwithan8 closed 1 month ago

nwithan8 commented 1 month ago

Description

Unhandled recursion during disposal of a Client or service would cause a stack overflow.

Closes #587

Testing

Add unit tests to verify disposal does not cause stack overflow

Pull Request Type

Please select the option(s) that are relevant to this PR.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Improvement (fixing a typo, updating readme, renaming a variable name, etc)