FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

upgrade jackson by boulter · Pull Request #1051 · HubSpot/jinjava · GitHub

upgrade jackson - #1051

Merged
boulter merged 5 commits into
masterfrom
boulter-patch-1
Jun 9, 2023
Merged

upgrade jackson#1051
boulter merged 5 commits into
masterfrom
boulter-patch-1

Conversation

boulter commented Apr 14, 2023

Copy link
Copy Markdown
Contributor

Upgrades to avoid CVE mentioned in #1050

boulter requested a review from jasmith-hs April 28, 2023 19:50

jasmith-hs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Can you make sure that this logic still works:

if (e instanceof LengthLimitingJsonProcessingException) {
throw new OutputTooBigException(
((LengthLimitingJsonProcessingException) e).getMaxSize(),
((LengthLimitingJsonProcessingException) e).getAttemptedSize()
);
}

boulter requested a review from jasmith-hs May 1, 2023 18:31

jowiho commented Jun 9, 2023

Copy link
Copy Markdown

In the meantime Jackson 2.15.2 is the latest stable version. I've confirmed that this PR builds and tests correctly with that Jackson version as well.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL