Skip to content

Resolve "500 Error viewing a single large commit"

Jerry Seto requested to merge 342703-500-error-viewing-a-single-large-commit into master

What does this MR do and why?

Avoid loading entire blob without limit for displaying diffs

Contributes to: #342703 (closed)

Changelog: fixed

How to set up and validate locally

  1. Push a commit containing a large file (7 GB was sufficiently large for me in development a smaller file might do as well)
  2. Enable the disable_load_entire_blob_for_diff_viewer feature flag
  3. Visit the commit page (e.g http://127.0.0.1:3000/root/testing-big-blog/-/commit/4df831b88c77136db133c604b6fa3d148db11157)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #342703 (closed)

Edited by Jerry Seto

Merge request reports