Skip to content

[O2B-1636] Limit the size of the log content parsed to build the log snippet - #2300

Merged
graduta merged 2 commits into
mainfrom
fix/O2B-1636/limit-log-snippet-parsing
Oct 26, 2026
Merged

graduta merged 2 commits into
mainfrom
fix/O2B-1636/limit-log-snippet-parsing

Conversation

@graduta

@graduta graduta commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • The log overview can no longer be slowed down or frozen by a log with a huge content on a single line

Notable changes for developers:

  • formatLogSnippet (added in [O2B-1636] Display snippet of content and allow users to click on row on log-overivew #2284) strips markdown with regular expressions that are quadratic on long lines (for example __a b repeated: about 2s for 192KB, and log texts have no maximum length). Only the beginning of the content can end up in the snippet, so only its first 2000 characters are now parsed
  • A unit test checks that a huge content is processed in less than 500ms. The formatLogSnippet unit tests were run locally and pass; the rest of the suite was not run locally

Changes made to the database:

  • None

@graduta
graduta requested a review from isaachilly as a code owner October 26, 2026 15:57
@graduta graduta added bug Something isn't working frontend test Label to be used when PR is only about tests labels Oct 26, 2026
@graduta graduta self-assigned this Oct 26, 2026
@codecov

codecov Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.00%. Comparing base (cf7cb88) to head (eb7bd5f).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...ib/public/utilities/formatting/formatLogSnippet.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2300      +/-   ##
==========================================
- Coverage   46.01%   46.00%   -0.01%     
==========================================
  Files        1070     1070              
  Lines       17609    17610       +1     
  Branches     3261     3261              
==========================================
  Hits         8102     8102              
- Misses       9507     9508       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@graduta
graduta merged commit bd85e9e into main Oct 26, 2026
24 of 26 checks passed
@graduta
graduta deleted the fix/O2B-1636/limit-log-snippet-parsing branch October 26, 2026 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend test Label to be used when PR is only about tests

Development

Successfully merging this pull request may close these issues.

1 participant