update activemq to minimize vulnerabilities#3637
update activemq to minimize vulnerabilities#3637derrickaw wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the ActiveMQ dependency to a newer version to mitigate known security vulnerabilities. The change includes updated documentation within the configuration file to reflect the current messaging API compatibility status and outlines the necessary steps for future upgrades. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3637 +/- ##
=============================================
+ Coverage 37.64% 52.30% +14.65%
- Complexity 614 6145 +5531
=============================================
Files 207 1054 +847
Lines 14226 63563 +49337
Branches 1451 6979 +5528
=============================================
+ Hits 5355 33244 +27889
- Misses 8400 28064 +19664
- Partials 471 2255 +1784
🚀 New features to boost your workflow:
|
|
Hi @Abacn, I am guessing that this upgrade is causing the Java PR tests to run beyond the 4 hour limit which seems like a regression of ~73 minutes to 240 minutes to be not a good move for us at this time. I am going to postpone this PR or even close it and redo it for 6.2.x once Beam 2.73 comes in. Sound good? Thanks. |
Uh oh!
There was an error while loading. Please reload this page.