Skip to content

Conversation

@julianladisch
Copy link
Contributor

@julianladisch julianladisch commented Dec 16, 2025

https://folio-org.atlassian.net/browse/MODINV-1321

Fix Netty CRLF injection request smuggling

Approach

Bump Vertx from 4.5.13 to 4.5.23.

This requires to also bump mod-source-record-storage-client from
5.10.0 to 5.10.14 to get a compatible RMB version, see
https://github.com/folio-org/raml-module-builder/releases/tag/v35.4.2

In TenantApiTest#shouldNotDropSchemaWithIncorrectConnectionOptions we
need an invalid port number because the new Vertx version makes longer
lookup retries for the hostname invalid that exceeds our timeout of
10 seconds. The invalid port number immediately fails and speeds up the
test.

@julianladisch julianladisch requested a review from a team December 16, 2025 17:04
@julianladisch julianladisch marked this pull request as draft December 17, 2025 11:47
https://folio-org.atlassian.net/browse/MODINV-1321

Fix Netty CRLF injection request smuggling

* CVE-2025-67735 GHSA-84h7-rjj3-6jx4

Approach

Bump Vertx from 4.5.13 to 4.5.23.

This requires to also bump mod-source-record-storage-client from
5.10.0 to 5.10.14 to get a compatible RMB version, see
https://github.com/folio-org/raml-module-builder/releases/tag/v35.4.2

In TenantApiTest#shouldNotDropSchemaWithIncorrectConnectionOptions we
need an invalid port number because the new Vertx version makes longer
lookup retries for the hostname `invalid` that exceeds our timeout of
10 seconds. The invalid port number immediately fails and speeds up the
test.
@sonarqubecloud
Copy link

@julianladisch julianladisch marked this pull request as ready for review January 11, 2026 11:27
@julianladisch julianladisch requested review from a team and Aliaksandr-Fedasiuk January 11, 2026 11:27
@julianladisch julianladisch merged commit cc4802a into master Jan 13, 2026
6 checks passed
@julianladisch julianladisch deleted the MODINV-1321 branch January 13, 2026 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants