Skip to content

[release/5.0] Use ArrayPool as default pool fallback #36559

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

sebastienros
Copy link
Member

@sebastienros sebastienros commented Sep 15, 2021

Backport of #35719
Fixes #30364

Customer Impact

Users can have their logs fill-up with exception messages when connections are closed by the client.

Testing

Functional tests
Unit tests

Risk

None known.

@ghost ghost added the area-runtime label Sep 15, 2021
@sebastienros sebastienros changed the title Use ArrayPool as default pool fallback [release/5.0] Use ArrayPool as default pool fallback Sep 15, 2021
@sebastienros
Copy link
Member Author

Waiting for tactics approval

@Pilchie Pilchie added the Servicing-consider Shiproom approval is required for the issue label Sep 16, 2021
@ghost
Copy link

ghost commented Sep 16, 2021

"Hi sebastienros. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@sebastienros
Copy link
Member Author

Got approval from tactics. Do I just merge it?

@halter73 halter73 added Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Sep 17, 2021
@halter73 halter73 merged commit d5ae64d into release/5.0 Sep 17, 2021
@halter73 halter73 deleted the sebros/bpallocs branch September 17, 2021 23:35
@dougbu dougbu added this to the 5.0.11 milestone Oct 5, 2021
Elanis pushed a commit to Elanis/portfolio that referenced this pull request Dec 13, 2022
Bumps [Microsoft.AspNetCore.TestHost](https://github.com/dotnet/aspnetcore) from 5.0.10 to 5.0.11.
<details>
<summary>Commits</summary>

- [`2924ca2`](dotnet/aspnetcore@2924ca2) Merged PR 18513: [internal/release/5.0] Update dependencies from dnceng/inter...
- [`eeb31f6`](dotnet/aspnetcore@eeb31f6) Merged PR 18423: [internal/release/5.0] Update dependencies from dnceng/inter...
- [`9035164`](dotnet/aspnetcore@9035164) Merged PR 18406: Merge from github
- [`1a95562`](dotnet/aspnetcore@1a95562) Merge from github
- [`60819f7`](dotnet/aspnetcore@60819f7) [release/5.0] Include SharedFx bundle in Hosting Bundle, instead of SharedFx ...
- [`d5ae64d`](dotnet/aspnetcore@d5ae64d) [release/5.0] Use ArrayPool as default pool fallback ([#36559](dotnet/aspnetcore#36559))
- [`cf7a1d4`](dotnet/aspnetcore@cf7a1d4) Fix feeds ([#36587](dotnet/aspnetcore#36587))
- [`f5b309d`](dotnet/aspnetcore@f5b309d) Merge pull request [#36516](dotnet/aspnetcore#36516) from vseanreesermsft/internal-merge-5.0-2021-09-14-...
- [`0366ec0`](dotnet/aspnetcore@0366ec0) Update SiteExtension package to 3.1.19-servicing-21417-13
- [`63e6c4e`](dotnet/aspnetcore@63e6c4e) Update to 5.0.207 SDK and 2.1.30 & 3.1.19 runtimes
- Additional commits viewable in [compare view](dotnet/aspnetcore@v5.0.10...v5.0.11)
</details>

<br />

Reviewed-on: https://gitea.dysnomia.studio/elanis/portfolio/pulls/14
Co-authored-by: elanis <elanis@noreply.example.org>
Co-committed-by: elanis <elanis@noreply.example.org>
Elanis pushed a commit to Dysnomia-Studio/dehash-me that referenced this pull request Mar 12, 2023
Bumps [Microsoft.AspNetCore.TestHost](https://github.com/dotnet/aspnetcore) from 5.0.10 to 5.0.11.
<details>
<summary>Commits</summary>

- [`2924ca2`](dotnet/aspnetcore@2924ca2) Merged PR 18513: [internal/release/5.0] Update dependencies from dnceng/inter...
- [`eeb31f6`](dotnet/aspnetcore@eeb31f6) Merged PR 18423: [internal/release/5.0] Update dependencies from dnceng/inter...
- [`9035164`](dotnet/aspnetcore@9035164) Merged PR 18406: Merge from github
- [`1a95562`](dotnet/aspnetcore@1a95562) Merge from github
- [`60819f7`](dotnet/aspnetcore@60819f7) [release/5.0] Include SharedFx bundle in Hosting Bundle, instead of SharedFx ...
- [`d5ae64d`](dotnet/aspnetcore@d5ae64d) [release/5.0] Use ArrayPool as default pool fallback ([#36559](dotnet/aspnetcore#36559))
- [`cf7a1d4`](dotnet/aspnetcore@cf7a1d4) Fix feeds ([#36587](dotnet/aspnetcore#36587))
- [`f5b309d`](dotnet/aspnetcore@f5b309d) Merge pull request [#36516](dotnet/aspnetcore#36516) from vseanreesermsft/internal-merge-5.0-2021-09-14-...
- [`0366ec0`](dotnet/aspnetcore@0366ec0) Update SiteExtension package to 3.1.19-servicing-21417-13
- [`63e6c4e`](dotnet/aspnetcore@63e6c4e) Update to 5.0.207 SDK and 2.1.30 & 3.1.19 runtimes
- Additional commits viewable in [compare view](dotnet/aspnetcore@v5.0.10...v5.0.11)
</details>

<br />

Reviewed-on: https://gitea.dysnomia.studio/elanis/dehash-me/pulls/21
Co-authored-by: elanis <elanis@noreply.example.org>
Co-committed-by: elanis <elanis@noreply.example.org>
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
Elanis added a commit to Dysnomia-Studio/dysnomia-website that referenced this pull request Jul 14, 2023
Bumps [Microsoft.AspNetCore.TestHost](https://github.com/dotnet/aspnetcore) from 5.0.10 to 5.0.11.
<details>
<summary>Commits</summary>

- [`2924ca2`](dotnet/aspnetcore@2924ca2) Merged PR 18513: [internal/release/5.0] Update dependencies from dnceng/inter...
- [`eeb31f6`](dotnet/aspnetcore@eeb31f6) Merged PR 18423: [internal/release/5.0] Update dependencies from dnceng/inter...
- [`9035164`](dotnet/aspnetcore@9035164) Merged PR 18406: Merge from github
- [`1a95562`](dotnet/aspnetcore@1a95562) Merge from github
- [`60819f7`](dotnet/aspnetcore@60819f7) [release/5.0] Include SharedFx bundle in Hosting Bundle, instead of SharedFx ...
- [`d5ae64d`](dotnet/aspnetcore@d5ae64d) [release/5.0] Use ArrayPool as default pool fallback ([#36559](dotnet/aspnetcore#36559))
- [`cf7a1d4`](dotnet/aspnetcore@cf7a1d4) Fix feeds ([#36587](dotnet/aspnetcore#36587))
- [`f5b309d`](dotnet/aspnetcore@f5b309d) Merge pull request [#36516](dotnet/aspnetcore#36516) from vseanreesermsft/internal-merge-5.0-2021-09-14-...
- [`0366ec0`](dotnet/aspnetcore@0366ec0) Update SiteExtension package to 3.1.19-servicing-21417-13
- [`63e6c4e`](dotnet/aspnetcore@63e6c4e) Update to 5.0.207 SDK and 2.1.30 & 3.1.19 runtimes
- Additional commits viewable in [compare view](dotnet/aspnetcore@v5.0.10...v5.0.11)
</details>

<br />

Co-authored-by: Elanis <elanis@hotmail.com>
Reviewed-on: https://gitea.dysnomia.studio/elanis/dysnomia-website/pulls/17
Co-authored-by: elanis <elanis@noreply.example.org>
Co-committed-by: elanis <elanis@noreply.example.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions Servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants