Skip to content

feat: migrate vertx 4 to 5#678

Open
Sunil-SKS wants to merge 7 commits intoSAP:mainfrom
Sunil-SKS:feat/migrateVertx5
Open

feat: migrate vertx 4 to 5#678
Sunil-SKS wants to merge 7 commits intoSAP:mainfrom
Sunil-SKS:feat/migrateVertx5

Conversation

@Sunil-SKS
Copy link
Contributor

@Sunil-SKS Sunil-SKS commented Feb 11, 2026

Latest Vertx v 5.0.7

@Sunil-SKS Sunil-SKS requested a review from a team as a code owner February 11, 2026 08:24
@Sunil-SKS
Copy link
Contributor Author

This PR still in work in progress [WIP]

@Sunil-SKS Sunil-SKS changed the title feat: Migrate vertx to latest version 5.0 feat: migrate vertx 4 to latest version vertx 5.0 Feb 11, 2026
@Sunil-SKS Sunil-SKS force-pushed the feat/migrateVertx5 branch 7 times, most recently from 22b18ce to 37864c4 Compare February 26, 2026 08:38
@cla-assistant
Copy link

cla-assistant bot commented Feb 26, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Sunil-SKS
❌ halber
You have signed the CLA already but the status is still pending? Let us recheck it.

@Sunil-SKS Sunil-SKS force-pushed the feat/migrateVertx5 branch 2 times, most recently from be5de9c to 3a625e1 Compare March 2, 2026 11:48
@Sunil-SKS Sunil-SKS changed the title feat: migrate vertx 4 to latest version vertx 5.0 feat: migrate vertx 4 to 5 Mar 2, 2026
@Sunil-SKS Sunil-SKS force-pushed the feat/migrateVertx5 branch from 3a625e1 to 21327da Compare March 2, 2026 11:54
Sunil-SKS and others added 6 commits March 2, 2026 12:56
Latest Vertx v 5.0.7
The Future implementation has been removed from PendingDeployment to support the migration from Vert.x 4 to Vert.x 5. The PendingDeployment class now returns objects directly, rather than Futures.
logback v 1.5.19
commons-io:commons v io:2.14.0
@Sunil-SKS Sunil-SKS force-pushed the feat/migrateVertx5 branch from 21327da to 78aef67 Compare March 2, 2026 11:56
If the BasicAuth added in the config then open apiendpoint will validate and adding BasicAuth is mandatory. As we send the empty basic auth this can be removed
@Sunil-SKS Sunil-SKS force-pushed the feat/migrateVertx5 branch from 78aef67 to f63f597 Compare March 3, 2026 10:15
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

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.

2 participants