updated spring dependencies - #329
Conversation
Codecov Report
@@ Coverage Diff @@
## master #329 +/- ##
============================================
+ Coverage 58.86% 58.92% +0.05%
Complexity 1206 1206
============================================
Files 131 131
Lines 5066 5066
Branches 593 593
============================================
+ Hits 2982 2985 +3
+ Misses 1809 1806 -3
Partials 275 275
Continue to review full report at Codecov.
|
| @@ -9,11 +9,11 @@ dependencies { | |||
| api("org.aspectj:aspectjrt:1.8.11") | |||
|
|
|||
| // TODO(anuraaga): Remove most of these? Seems only Configurable annotation is used | |||
There was a problem hiding this comment.
@anuraaga 's comment hints that these dependencies could be removed. If it is not a big effort, maybe this is a good time to do so?
Otherwise LGTM.
There was a problem hiding this comment.
Yeah Good callout. I think it's a small change relatively but in order to ensure nothing breaks and SDK works as expected we might need to do some additional testing so I have scoped that work out of this PR if that makes sense.
Issue #, if available:
Spring RCE issue: https://tanzu.vmware.com/security/cve-2022-22965
Description of changes:
Updated spring dependencies
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.