Skip to content
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

Bump Kafka dependencies #277

Merged
merged 1 commit into from
Dec 4, 2024
Merged

Bump Kafka dependencies #277

merged 1 commit into from
Dec 4, 2024

Conversation

yeikel
Copy link
Contributor

@yeikel yeikel commented Oct 10, 2024

No description provided.

@yeikel
Copy link
Contributor Author

yeikel commented Oct 10, 2024

Opened it to see what the CI thinks but it seems that the workflow needs approval. Can you help here? @vietj

@ppatierno ppatierno marked this pull request as ready for review October 10, 2024 17:15
@vietj
Copy link
Contributor

vietj commented Oct 10, 2024

@yeikel CI does not seem happy

@yeikel
Copy link
Contributor Author

yeikel commented Oct 10, 2024

@yeikel CI does not seem happy

Thanks. I'll take it from here

@yeikel yeikel changed the title Bump Kafka to 3.8.0 Bump Kafka dependencies Nov 27, 2024
@yeikel yeikel marked this pull request as draft November 27, 2024 23:43
@yeikel yeikel marked this pull request as ready for review December 3, 2024 03:07
@yeikel
Copy link
Contributor Author

yeikel commented Dec 3, 2024

@vietj I reduced the scope of the PR to continue with 3.7.x for now while still fixing the active CVEs.

The upgrade beyond this needs more work that can be a follow up

@vietj vietj added this to the 5.0.0 milestone Dec 4, 2024
@vietj vietj merged commit c371510 into vert-x3:master Dec 4, 2024
1 check passed
@yeikel
Copy link
Contributor Author

yeikel commented Dec 4, 2024

@vietj Can we backport this to 4.x?

@tsegismont
Copy link
Contributor

@vietj Can we backport this to 4.x?

@ppatierno is that ok with you?

@ppatierno
Copy link
Member

Fine with me!

@tsegismont
Copy link
Contributor

Fine with me!

Thank you @ppatierno

Then please go ahead @yeikel !

yeikel added a commit to yeikel/vertx-kafka-client that referenced this pull request Jan 3, 2025
@yeikel yeikel mentioned this pull request Jan 3, 2025
@yeikel
Copy link
Contributor Author

yeikel commented Jan 3, 2025

Fine with me!

Thank you @ppatierno

Then please go ahead @yeikel !

@tsegismont @ppatierno See #283

And thank you!

ppatierno pushed a commit that referenced this pull request Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants