microsoft / AzUrlShortener

An simple and easy Url Shortener
MIT License
519 stars 561 forks source link

Using ReadToEndAsync to replace ReadToEnd #466

Closed catcherwong closed 1 year ago

catcherwong commented 1 year ago

Based on Avoid using synchronous Read/Write overloads on HttpRequest.Body and HttpResponse.Body and many samples of azure function, such as

https://github.com/azure-samples/azure-sql-binding-func-dotnet-todo/blob/main/PostToDo.cs#L27

and

https://github.com/azure-samples/media-services-v3-dotnet-core-functions-integration/blob/main/Functions/CreateEmptyAsset.cs#L86

Using ReadToEndAsync to replace ReadToEnd may be better here.

pjirsa commented 1 year ago

@all-contributors add @catcherwong for code

allcontributors[bot] commented 1 year ago

@pjirsa

This project's configuration file has malformed JSON: .all-contributorsrc. Error:: Unexpected token } in JSON at position 4654

pjirsa commented 1 year ago

@all-contributors add @catcherwong for code

allcontributors[bot] commented 1 year ago

@pjirsa

I've put up a pull request to add @catcherwong! :tada: