search

GoogleCloudPlatform / datacatalog-connectors-hive

Sample code with integration between Data Catalog and Hive data source.
Apache License 2.0
25 stars 14 forks source link

Update log4j library #32

Closed bonifacyj closed 2 years ago

bonifacyj commented 2 years ago

- What I did

Updated log4j library to mitigate vulnerability found in earlier version: https://logging.apache.org/log4j/2.x/security.html

- How I did it

Changed version in pom file

mesmacosta commented 2 years ago

Hi @bonifacyj is this PR related to an issue? If so could you provide additional context about the issue on the PR?

In the meanwhile I am checking why the Python CI is breaking, which is unrelated to your change.

Thanks

bonifacyj commented 2 years ago

Hi @bonifacyj is this PR related to an issue? If so could you provide additional context about the issue on the PR?

In the meanwhile I am checking why the Python CI is breaking, which is unrelated to your change.

Thanks

Hi Marcelo! Thanks for having a look. I've updated my PR description to explain what issue it tries to fix.

mesmacosta commented 2 years ago

@bonifacyj the breaking CI has been fixed, can you merge the master branch in your bonifacyj:patch-1 branch?

mesmacosta commented 2 years ago

@bonifacyj I've mirrored your changes in the PR #34 so the CI runs with the up to date code, adding you as co-author of the changes.