Skip to content

[pull] main from spring-projects:main#705

Merged
pull[bot] merged 7 commits into
deepharness:mainfrom
spring-projects:main
May 31, 2026
Merged

[pull] main from spring-projects:main#705
pull[bot] merged 7 commits into
deepharness:mainfrom
spring-projects:main

Conversation

@pull

@pull pull Bot commented May 31, 2026

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

SebTardif and others added 7 commits May 31, 2026 16:13
InspectingOutputStream extends OutputStream but does not override
close(). When content exceeds MEMORY_LIMIT (approximately 4 KB),
convertToTempFile() replaces the ByteArrayOutputStream delegate with a
FileOutputStream. The try-with-resources in InspectedContent.of() calls
close(), which inherits the no-op OutputStream.close(), leaving the
FileOutputStream open and leaking a file descriptor.

Override close() to delegate to the underlying stream.

See gh-50639

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
* close-inspecting-output-stream:
  Close FileOutputStream delegate in InspectingOutputStream

Closes gh-50639
The RSocket WebSocket transport's HttpServerSslCustomizer kept its own
copy of the SNI provider lookup that returned null for server names
without a mapped SSL bundle. This commit adapts the copy to fall back
to the default provider, as it is done in gh-50301.

See gh-50640

Signed-off-by: Lee JiWon <dlwldnjs1009@gmail.com>
* rsocket-sni-fallback:
  Fall back to default SSL provider for unmapped SNI

Closes gh-50640
@pull pull Bot locked and limited conversation to collaborators May 31, 2026
@pull pull Bot added the ⤵️ pull label May 31, 2026
@pull pull Bot merged commit 451c873 into deepharness:main May 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants