diff options
| author | Boyan Tabakov <boyan.tabakov@futurice.com> | 2013-09-04 10:33:09 +0300 |
|---|---|---|
| committer | Boyan Tabakov <boyan.tabakov@futurice.com> | 2013-09-07 14:44:03 +0300 |
| commit | 01ff084a4df76ba0856a513aca9bdf8f1d550365 (patch) | |
| tree | d90ef89169f4e53dc702172b3f3b01a030619549 /features/project/commits | |
| parent | 71d31a38fc73252a76076820c63d054a8047d667 (diff) | |
| download | gitlab-ce-01ff084a4df76ba0856a513aca9bdf8f1d550365.tar.gz | |
Improved large commit handling.
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.
Diffstat (limited to 'features/project/commits')
| -rw-r--r-- | features/project/commits/commits.feature | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/features/project/commits/commits.feature b/features/project/commits/commits.feature index 56069cdc977..d962471ebdb 100644 --- a/features/project/commits/commits.feature +++ b/features/project/commits/commits.feature @@ -27,3 +27,11 @@ Feature: Project Browse commits Scenario: I browse commits stats Given I visit my project's commits stats page Then I see commits stats + + Scenario: I browse big commit + Given I visit big commit page + Then I see big commit warning + + Scenario: I browse huge commit + Given I visit huge commit page + Then I see huge commit message |