[BUG] Use of Task.Result directly blocks thread

[rfriend-zilo]

Bug Report Checklist

• Have you provided a full/minimal spec to reproduce the issue?
• Have you validated the input using an OpenAPI validator (example)?
• [ x] Have you tested with the latest master to confirm the issue still exists?
• [x ] Have you searched for related issues/PRs?
• What's the actual output vs expected output?
• [Optional] Sponsorship to speed up the bug fix or feature request (example)

Description

The direct call here to Task.Result is blocking the thread, and when used in a blazor application prevents UI from rendering :

openapi-generator/modules/openapi-generator/src/main/resources/csharp/ApiClient.mustache

Line 628 in 9ae8775

return action().Result;

Related issues/PRs

https://github.com/OpenAPITools/openapi-generator/pull/18915/files/0639c25fe8affd4c6ad8ea3d26711cc0f83a7bf1

Suggest a fix

Rather than calling async code from sync code using .Result, inside ExecAsync just await action() then construct Func<RestClient, RestResponse> getResponse using the result from that awaited call.

[wing328]

Rather than calling async code from sync code using .Result, inside ExecAsync just await action() then construct Func<RestClient, RestResponse> getResponse using the result from that awaited call.

can you please file a PR with the suggested fix when you've time?

[rfriend-zilo]

My solution was not correct, my suggestion is to revert the changes in the PR.

[wing328]

@filipe-silva can you please review this issue reported by @rfriend-zilo when you've time?

[filipe-silva]

@filipe-silva can you please review this issue reported by @rfriend-zilo when you've time?

This concern makes sense, I did a fix to resolve this issue.
How do I merge it or include it in this PR?

[wing328]

@filipe-silva can you please file a new PR?

[wing328]

@rfriend-zilo can you please review the fix when you've time?

#19231