fix: deadlocks in client applications

[DouglasHammon-FV]

This PR

-Added configure await false to avoid deadlocks when executing synchronously from client applications.

Related Issues

Fixes #149

Notes

I published a Nuget package locally and confirmed this fixes the deadlock issue that I'm seeing in my project.

[codecov]

Codecov Report

Merging #150 (e261d9b) into main (598270d) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #150   +/-   ##
=======================================
  Coverage   93.91%   93.91%           
=======================================
  Files          20       20           
  Lines         542      542           
  Branches       39       39           
=======================================
  Hits          509      509           
  Misses         20       20           
  Partials       13       13           

Files Changed	Coverage Δ
src/OpenFeature/OpenFeatureClient.cs	96.10% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[benjiro]

LGTM. Just need to rebase and sign your commit https://github.com/open-feature/dotnet-sdk/pull/150/checks?check_run_id=16907801522

[toddbaert]

LGTM. Just need to rebase and sign your commit https://github.com/open-feature/dotnet-sdk/pull/150/checks?check_run_id=16907801522

Ya @DouglasHammon-FV sorry for the tedium, but this is a CNCF requirement.

Thanks for the quick fix!

[toddbaert]

I've updated the title a bit (it's included in release notes, as well as a link to this PR).

[DouglasHammon-FV]

@toddbaert I have signed the commit. But code coverage is failing now. Looks like an issue with uploading stuff. I don't know how to kickoff that again (manually). Please review.

[toddbaert]

@toddbaert I have signed the commit. But code coverage is failing now. Looks like an issue with uploading stuff. I don't know how to kickoff that again (manually). Please review.

This is a transient error. I re-ran and it worked. merging now!

[DouglasHammon-FV]

@toddbaert Thanks! When can I expect a new release with these changes?

[toddbaert]

@toddbaert Thanks! When can I expect a new release with these changes?

Shortly after the automated release PR is merged. Since this is the only change I will do so as soon as the CI passes..

[beeme1mr]

Thanks for reporting (and fixing) this issue @DouglasHammon-FV!