Skip to content

Commit

Permalink
This updates Jackson to 2.16.1. This version does include a change to…
Browse files Browse the repository at this point in the history
… redact input data in exceptions. So a couple tests needed modifications. Additionally, this uses ion-java 1.10.5 which has a fix for CVE-2024-21634, so this PR will resolve #3926. (#4134)

Signed-off-by: David Venable <[email protected]>
  • Loading branch information
dlvenable authored Feb 28, 2024
1 parent bb121e8 commit 1cbfdd1
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ subprojects {
}
}
dependencies {
implementation platform('com.fasterxml.jackson:jackson-bom:2.15.3')
implementation platform('com.fasterxml.jackson:jackson-bom:2.16.1')
implementation platform('org.eclipse.jetty:jetty-bom:9.4.53.v20231009')
implementation platform('io.micrometer:micrometer-bom:1.10.5')
implementation libs.guava.core
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ void constructor_throws_if_schema_is_invalid() {
RuntimeException actualException = assertThrows(RuntimeException.class, this::createObjectUnderTest);

assertThat(actualException.getMessage(), notNullValue());
assertThat(actualException.getMessage(), containsString(invalidSchema));
assertThat(actualException.getMessage(), containsString("was expecting comma"));
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,6 @@ void constructor_throws_if_schema_is_invalid() {
RuntimeException actualException = assertThrows(RuntimeException.class, this::createObjectUnderTest);

assertThat(actualException.getMessage(), notNullValue());
assertThat(actualException.getMessage(), containsString(invalidSchema));
assertThat(actualException.getMessage(), containsString("was expecting comma"));
}

Expand Down

0 comments on commit 1cbfdd1

Please sign in to comment.