Skip to content

fix: overwrite of known size after Builder.stream generator execution - #14541

Merged
algebraic-dev merged 1 commit into
masterfrom
sofia/async-http-fix-body-stream
Jul 30, 2026
Merged

fix: overwrite of known size after Builder.stream generator execution#14541
algebraic-dev merged 1 commit into
masterfrom
sofia/async-http-fix-body-stream

Conversation

@algebraic-dev

@algebraic-dev algebraic-dev commented Jul 24, 2026

Copy link
Copy Markdown
Member

This PR fixes a possible time-sensitive overwrite of the known size by the Builder.stream functions.

@algebraic-dev algebraic-dev self-assigned this Jul 24, 2026
@algebraic-dev
algebraic-dev requested a review from TwoFX as a code owner July 24, 2026 17:38
@github-actions github-actions Bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Jul 24, 2026
@mathlib-lean-pr-testing

Copy link
Copy Markdown

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 366c0a8ba4f6d6445c8f9383c4fab0a4be1cbc9d --onto 3259610687883ec1ea48c481aba2469f2f83facf. You can force Mathlib CI using the force-mathlib-ci label. (2026-07-24 17:58:34)

@leanprover-bot

Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 366c0a8ba4f6d6445c8f9383c4fab0a4be1cbc9d --onto 49ff95727f98d43984726b26742d17a1ceea9dd5. You can force reference manual CI using the force-manual-ci label. (2026-07-24 17:58:35)

@algebraic-dev algebraic-dev changed the title fix: overwrite of known size after Builder.stream generator e… fix: overwrite of known size after Builder.stream generator execution Jul 25, 2026
@ecyrbe

ecyrbe commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

mentioning corresponding issue : #14527

@algebraic-dev
algebraic-dev added this pull request to the merge queue Jul 30, 2026
Merged via the queue into master with commit 44d40a3 Jul 30, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-library Library toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants