-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
jakarta libs upgrade #76
base: master
Are you sure you want to change the base?
Conversation
build.gradle
Outdated
// resolutionStrategy.eachDependency { details -> | ||
// if (details.requested.group == 'io.netty') { | ||
// details.useVersion "4.1.77.Final" | ||
// } |
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.
please delete commented code
build.gradle
Outdated
implementation (group: 'com.fasterxml.jackson.core', name: 'jackson-core', version: versions.jackson) { | ||
force = true | ||
} | ||
implementation (group: 'com.fasterxml.jackson.core', name: 'jackson-databind', version: '2.15.2') |
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.
can't we use 2.13.2 ( version: versions.jackson))
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.
local dependency check scan flags jackson-databind 2.13.2 as vulnerable to CVE-2020-36518, CVE-2022-42003, CVE-2022-42004
build.gradle
Outdated
implementation 'net.logstash.logback:logstash-logback-encoder:6.4' | ||
|
||
implementation group: 'com.microsoft.azure', name: 'azure-storage-blob', version: '11.0.0' | ||
implementation group: 'com.azure', name: 'azure-storage-blob', version: '12.18.0' |
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.
latest is 12.23.0, better to use latest version, if it is compatable
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.
Minor comments
# Conflicts: # build.gradle
Before creating a pull request make sure that:
Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/DTSRD-72
Change description
Does this PR introduce a breaking change? (check one with "x")