Skip to content

* GenericActionPipelinePolicy - ConfigureAwait(false) for ProcessAsync method - #353

Merged
joseharriaga merged 1 commit into
openai:mainfrom
vanek021:main
Feb 19, 2025
Merged

joseharriaga merged 1 commit into
openai:mainfrom
vanek021:main

Conversation

@vanek021

Copy link
Copy Markdown
Contributor

If this was not intentional, it might be better to add ConfigureAwait(false) to this method. In my case, there was dedlock with Blazor Server.
I think it would be better if the synchronization context was not captured inside the library

@joseharriaga joseharriaga self-assigned this Feb 19, 2025
@joseharriaga
joseharriaga merged commit 3d7d89d into openai:main Feb 19, 2025
@joseharriaga

Copy link
Copy Markdown
Collaborator

Thank you for your contribution, @vanek021 ! Great catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants