Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

Use ReadFormAsync instead Form on HttpRequest #623

Closed
sbebrys opened this issue Dec 21, 2016 · 5 comments
Closed

Use ReadFormAsync instead Form on HttpRequest #623

sbebrys opened this issue Dec 21, 2016 · 5 comments
Assignees
@brockallen
Labels
enhancement
Milestone
RTM

Comments

@sbebrys
Copy link

sbebrys commented Dec 21, 2016

Base on performance test, which showed hot path in using Form synch property (what in fact is sync-over-async) can You use async equivalent in this place - ReadFormAsync.
Check this comment on sync action: FormFeature.
@brockallen brockallen self-assigned this Dec 21, 2016
@brockallen
Copy link
Member

look ok? 43849dd

@brockallen brockallen added this to the RTM milestone Dec 22, 2016
@sbebrys
Copy link
Author

sbebrys commented Dec 22, 2016

Great job!

@brockallen
Copy link
Member

@leastprivilege any objections?

@brockallen
Copy link
Member

merged. thanks for the heads up!

@lock
Copy link

lock bot commented Jan 15, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@brockallen brockallen

Labels
enhancement
Projects
None yet
Milestone
RTM
Development

No branches or pull requests
3 participants
@brockallen @leastprivilege @sbebrys