-
Notifications
You must be signed in to change notification settings - Fork 23
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
Upgrade quartz dependency #489
Upgrade quartz dependency #489
Conversation
Kudos, SonarCloud Quality Gate passed! |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #489 +/- ##
=========================================
Coverage 90.37% 90.37%
Complexity 92 92
=========================================
Files 12 12
Lines 343 343
Branches 56 56
=========================================
Hits 310 310
Misses 29 29
Partials 4 4 ☔ View full report in Codecov by Sentry. |
Can we use rc1 dependencies in our libs? |
We are using 2.3.2. Quartz's next major version lib hasn't been released yet. Updated versions are in RC, and the issue suggests the version in RC as well. @daneshk @keizer619 Please give your suggestion. |
According to the quartz issue thread[1], this vulnerability does impact only if we expose the connection factory to be configured by the user. otherwise, there is no way to be vulnerable to the issue. They are still discussing whether it should handled at the application level. IMO, we are not impacted by this. and we can't go with rc1 release. |
If we are not impacted, i guess we can ignore the vulnerability since we cannot use rc1 builds as well |
Purpose
$Subject
Fixes: https://github.com/wso2-enterprise/internal-support-ballerina/issues/437
Examples
Checklist