confluentinc / kafka-connect-hdfs

Kafka Connect HDFS connector
Other
9 stars 396 forks source link

CC-18964 CCMSG-2421 CCMSG-2420 Update Ivy, commons-net, json-smart, mina-core and xmlbuilder #652

Closed venkatteki closed 1 year ago

venkatteki commented 1 year ago

Problem

woodstox-core:5.3.0 and jettison:1.1 are vulnerable.

Solution

Pin woodstox-core:6.5.0 and jettison:1.5.3 to fix the CVE

Does this solution apply anywhere else?
If yes, where?

Test Strategy

Testing done:

Release Plan

janjwerner-confluent commented 1 year ago

Two nits:

janjwerner-confluent commented 1 year ago

Thank you for the catch with the incorrect versions @naveenmall11

venkatteki commented 1 year ago

@confluentinc/connect-team1 can you please review this PR

naveenmall11 commented 1 year ago

Thank you for the catch with the incorrect versions @naveenmall11

@janjwerner-confluent can we confirm on twistlock that these dependencies are not there on this PR scan?

janjwerner-confluent commented 1 year ago

The latest build 18 takes down CVE count from 23 in master to 8 the remaining issues are connector breaking. https://twistlock.tools.confluent-internal.io/#!/monitor/vulnerabilities/images/ci?search=Confluent%20Public%20Repo%20PR%20builder%2Fkafka-connect-hdfs%2FPR-652 Please review / test /merge. @venkatteki @snehashisp

venkatteki commented 1 year ago

@confluentinc/connect-team1 Can you please review

rishabhbits038 commented 1 year ago

Verified the connector in kafka-docker-playground. Merging this