-
Notifications
You must be signed in to change notification settings - Fork 99
NIFIREG-387 - Upgraded spring boot and some additional depedencies re… #277
Conversation
…quired to allow this upgrade.
pom.xml
Outdated
<jackson.version>2.9.9</jackson.version> | ||
<jackson.databind.version>2.9.10.3</jackson.databind.version> | ||
<spring.boot.version>2.1.12.RELEASE</spring.boot.version> | ||
<spring.boot.version>2.2.6.RELEASE</spring.boot.version> | ||
<spring.security.version>5.1.8.RELEASE</spring.security.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like spring security can be bumped to along with the new version of spring-framework, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to 5.1.9.RELEASE.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @thenatog. Are any changes to our NOTICE file required? Not sure if any of these are called out by version number there.
pom.xml
Outdated
<jackson.version>2.9.9</jackson.version> | ||
<jackson.databind.version>2.9.10.3</jackson.databind.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like jackson can go to 2.10.3. also not sure why we are at separate versions for jackson and jackson databank
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to 2.10.3 and removed the jackson.databind.version parameter and changed all dependencies to use only jackson.version.
… versions. Updated NOTICE file.
Update the NOTICE file. |
@kevdoran did you need another look at this or were updating the |
Hey @alopresto @thenatog - sorry, this one fell off my radar. That was all I had. +1 from me. Thanks! |
…quired to allow this upgrade.