Skip to content

Commit

Permalink
Use explicit jackson-bom dependencies to control Jackson versions
Browse files Browse the repository at this point in the history
  • Loading branch information
emlun committed May 2, 2023
1 parent 57b28e4 commit 5a19899
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 3 deletions.
5 changes: 3 additions & 2 deletions NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,9 @@

Fixes:

* Removed version constraints on non-core Jackson modules; Jackson ships its own
BOM constraints to align module versions.
* Added explicit `dependencyManagement` dependencies on `jackson-bom`, and
replaced version constraints on other Jackson modules with version constraint
on `jackson-bom`.
* Fixed incompatibility with Jackson version 2.15.0-rc1 and later.
* Fixed linking issue when running in Java 8.

Expand Down
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ wrapper {
dependencies {
constraints {
api('com.augustcellars.cose:cose-java:[1.0.0,2)')
api('com.fasterxml.jackson.core:jackson-databind:[2.13.2.1,3)')
api('com.fasterxml.jackson:jackson-bom:[2.13.2.1,3)')
api('com.google.guava:guava:[24.1.1,32)')
api('com.upokecenter:cbor:[4.5.1,5)')
api('org.apache.httpcomponents.client5:httpclient5:[5.0.0,6)')
Expand Down
1 change: 1 addition & 0 deletions webauthn-server-attestation/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ dependencies {

api(project(":webauthn-server-core"))

implementation(platform("com.fasterxml.jackson:jackson-bom"))
implementation(project(":yubico-util"))
implementation("com.fasterxml.jackson.core:jackson-databind")
implementation("org.slf4j:slf4j-api")
Expand Down
1 change: 1 addition & 0 deletions webauthn-server-core/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ description = "Yubico WebAuthn server core API"
dependencies {
api(platform(rootProject))

implementation(platform("com.fasterxml.jackson:jackson-bom"))
implementation(project(":yubico-util"))
implementation("com.augustcellars.cose:cose-java")
implementation("com.fasterxml.jackson.core:jackson-databind")
Expand Down
1 change: 1 addition & 0 deletions yubico-util/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ dependencies {
api(platform(rootProject))

api("com.fasterxml.jackson.core:jackson-databind")
api(platform("com.fasterxml.jackson:jackson-bom"))

implementation("com.fasterxml.jackson.dataformat:jackson-dataformat-cbor")
implementation("com.fasterxml.jackson.datatype:jackson-datatype-jdk8")
Expand Down

1 comment on commit 5a19899

@github-actions
Copy link

Choose a reason for hiding this comment

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

Mutation test results

Package Coverage Stats Prev Prev
Overall 81 % 🔹 1265 🔻 / 1559 🔹 81 % 1267 / 1559
com.yubico.fido.metadata 68 % 🔹 217 🔹 / 318 🔹 68 % 217 / 318
com.yubico.internal.util 47 % 🔹 57 🔹 / 120 🔹 47 % 57 / 120
com.yubico.webauthn 87 % 🔹 558 🔹 / 637 🔹 87 % 558 / 637
com.yubico.webauthn.attestation 92 % 🔹 13 🔹 / 14 🔹 92 % 13 / 14
com.yubico.webauthn.data 93 % 🔹 395 🔻 / 423 🔹 93 % 397 / 423
com.yubico.webauthn.extension.appid 100 % 🏆 13 🔹 / 13 🔹 100 % 13 / 13
com.yubico.webauthn.extension.uvm 50 % 🔹 12 🔹 / 24 🔹 50 % 12 / 24
com.yubico.webauthn.meta 0 % 🔹 0 🔹 / 10 🔹 0 % 0 / 10

Previous run: 57b28e4 - Diff

Detailed reports: workflow run #216

Please sign in to comment.