Skip to content

[CTM-500] override all netty#4913

Merged
calypsomatic merged 3 commits into
developfrom
bm/ctm-500_netty2
Jun 16, 2026
Merged

[CTM-500] override all netty#4913
calypsomatic merged 3 commits into
developfrom
bm/ctm-500_netty2

Conversation

@calypsomatic

@calypsomatic calypsomatic commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Jira ticket: https://broadworkbench.atlassian.net/browse/CTM-500

Summary of changes

#4910 only overrode netty-codec-http. This PR overrides all the netty libraries to a safe version.

What

Why

Testing these changes

What to test

Who tested and where

  • This change is covered by automated tests
    • NB: Rerun automation tests on this PR by commenting jenkins retest or jenkins multi-test.
  • I validated this change
  • Primary reviewer validated this change
  • I validated this change in the dev environment

@calypsomatic calypsomatic changed the title override all netty [CTM-500] override all netty Jun 15, 2026
@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.07%. Comparing base (9fcabd8) to head (94a951f).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #4913   +/-   ##
========================================
  Coverage    74.07%   74.07%           
========================================
  Files          131      131           
  Lines        11101    11101           
  Branches       891      891           
========================================
  Hits          8223     8223           
  Misses        2878     2878           

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9fcabd8...94a951f. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@calypsomatic calypsomatic marked this pull request as ready for review June 15, 2026 15:29
@calypsomatic calypsomatic requested a review from a team as a code owner June 15, 2026 15:29
@calypsomatic calypsomatic requested a review from aednichols June 15, 2026 17:57
@calypsomatic calypsomatic merged commit 51f4642 into develop Jun 16, 2026
15 checks passed
@calypsomatic calypsomatic deleted the bm/ctm-500_netty2 branch June 16, 2026 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants