Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Release 0.9.2 #166

Merged
merged 2 commits into from
Aug 23, 2022
Merged

Release 0.9.2 #166

merged 2 commits into from
Aug 23, 2022

Conversation

spenes
Copy link
Contributor

@spenes spenes commented Aug 22, 2022

Snyk was reporting security vulnerabilities in the Jackson libraries that are brough transitively from other libraries.
In order to get rid of these vulnerabilities, we've added those libraries as dependency to project explicitly and bump their versions.

Snyk was reporting security vulnerabilities in the Jackson libraries that are brough transitively from other libraries.
In order to get rid of these vulnerabilities, we've added those libraries as dependency to project explicitly and bump their versions.
@spenes spenes changed the title Bump jackson libraries to 2.12.7 Release 0.9.2 Aug 22, 2022
Copy link

@benjben benjben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@@ -67,6 +67,8 @@ lazy val transformer = project.in(file("transformer"))

lazy val commonDependencies = Seq(
Dependencies.jacksonCbor,
Dependencies.jacksonDatabind,
Dependencies.jacksonScala,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the effect of adding jackson-module-scala to the deps? Given that jackson is a Java lib

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to this, it provides serializers and deserializers for Scala entities like case class, Map, Option etc.

It was brought implicitly previously. When I've bumped jackson databind library version to 2.12.7, it threw error due to incompatibility with jackson scala library. Therefore, I've included it too explicitly and bumped its version.

@spenes spenes merged commit 3a9bdbb into master Aug 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants