01ff084a4d
Previously, only number of changed files mattered. Now, number of lines to render in the diff are also taken into account. A hard limit is set, above which diffs are not rendered and users are not allowed to override that. This prevents high server resource usage with huge commits. Related to #1745, #2259 In addition, handle large commits for MergeRequests and Compare controllers. Also fixes a bug where diffs are loaded twice, if user goes directly to merge_requests/:id/diffs URL.
33 lines
857 B
Ruby
33 lines
857 B
Ruby
class CommitLoadContext < BaseContext
|
|
def execute
|
|
result = {
|
|
commit: nil,
|
|
suppress_diff: false,
|
|
line_notes: [],
|
|
notes_count: 0,
|
|
note: nil,
|
|
status: :ok
|
|
}
|
|
|
|
commit = project.repository.commit(params[:id])
|
|
|
|
if commit
|
|
line_notes = project.notes.for_commit_id(commit.id).inline
|
|
|
|
result[:commit] = commit
|
|
result[:note] = project.build_commit_note(commit)
|
|
result[:line_notes] = line_notes
|
|
result[:notes_count] = project.notes.for_commit_id(commit.id).count
|
|
|
|
begin
|
|
result[:suppress_diff] = true if commit.diff_suppress? && !params[:force_show_diff]
|
|
result[:force_suppress_diff] = commit.diff_force_suppress?
|
|
rescue Grit::Git::GitTimeout
|
|
result[:suppress_diff] = true
|
|
result[:status] = :huge_commit
|
|
end
|
|
end
|
|
|
|
result
|
|
end
|
|
end
|