summaryrefslogtreecommitdiff
path: root/app/services
Commit message (Collapse)AuthorAgeFilesLines
* Remove unnused escape_html methodosw-markdown-bypass-for-commit-messagesOswaldo Ferreira2018-02-021-4/+0
|
* Bypass commits title markdown on notesOswaldo Ferreira2018-02-021-4/+18
|
* Close and do not reload MR diffs when source branch is deletedOswaldo Ferreira2018-01-291-2/+14
|
* Merge branch '41532-email-reason' into 'master'Sean McGivern2018-01-192-34/+40
|\ | | | | | | | | | | | | Show why a notification email was sent Closes #41532 and #1366 See merge request gitlab-org/gitlab-ce!16160
| * Initial work to add notification reason to emailsMario de la Ossa2018-01-162-34/+40
| | | | | | | | | | | | | | | | | | | | | | Adds `#build_notification_recipients` to `NotificationRecipientService` that returns the `NotificationRecipient` objects in order to be able to access the new attribute `reason`. This new attribute is used in the different notifier methods in order to add the reason as a header: `X-GitLab-NotificationReason`. Only the reason with the most priority gets sent.
* | Merge branch 'feature/merge-request-system-hook' into 'master'Douwe Maan2018-01-183-2/+9
|\ \ | | | | | | | | | | | | System hooks for Merge Requests See merge request gitlab-org/gitlab-ce!14387
| * | use class reader instead of dynamic constantAlexis Reigel2018-01-171-1/+1
| | |
| * | use safer .hooks_for instead of .public_sendAlexis Reigel2018-01-171-1/+1
| | | | | | | | | | | | | | | with .public_send we can't make sure that the scope on the model actually exists.
| * | test hook for system merge request hookAlexis Reigel2018-01-041-0/+7
| | |
* | | Merge branch 'sh-backport-10-3-4-security-fixes' into 'master'Oswaldo Ferreira2018-01-173-8/+24
|\ \ \ | | | | | | | | | | | | | | | | Backport 10.3.4 security fixes into master See merge request gitlab-org/gitlab-ce!16509
| * | | Merge branch ↵Robert Speicher2018-01-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '41293-fix-command-injection-vulnerability-on-system_hook_push-queue-through-web-hook' into 'security-10-3' Don't allow line breaks on HTTP headers See merge request gitlab/gitlabhq!2277 (cherry picked from commit 7fc0a6fc096768a5604d6dd24d7d952e53300c82) 073b8f9c Don't allow line breaks on HTTP headers
| * | | Merge branch '41567-projectfix' into 'security-10-3'Sean McGivern2018-01-161-6/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | check project access on MR create See merge request gitlab/gitlabhq!2273 (cherry picked from commit 1fe2325d6ef2bced4c5e97b57691c894f38b2834) 43e85f49 check project access on MR create
| * | | Merge branch 'sh-validate-path-project-import-10-3' into 'security-10-3'Stan Hu2018-01-161-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Validate project path in Gitlab import - 10.3 port See merge request gitlab/gitlabhq!2268 (cherry picked from commit 94c82376d66fc80d46dd2d5eeb5bade408ec6a7e) 2b94a7c2 Validate project path in Gitlab import
* | | Merge branch '41727-target-branch-name' into 'master'Sean McGivern2018-01-171-0/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Set target_branch to the ref branch when creating MR from issue Closes #41727 See merge request gitlab-org/gitlab-ce!16422
| * | | Set target_branch to the ref branch when creating MR from issue41727-target-branch-nameJarka Kadlecová2018-01-171-0/+1
| | | |
* | | | Merge branch '37199-labels-fix' into 'master'Sean McGivern2018-01-171-1/+11
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Keep subscribers when promoting labels to group labels Closes #37199 See merge request gitlab-org/gitlab-ce!16343
| * | | Keep subscribers when promoting labels to group labels37199-labels-fixJarka Kadlecová2018-01-121-1/+11
| | | |
* | | | Merge branch 'issue-30101' into 'master'Rémy Coutable2018-01-161-15/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update 'removed assignee' note to include old assignee reference Closes #30101 See merge request gitlab-org/gitlab-ce!16301
| * | | | Update 'removed assignee' note to include old assignee referenceMaurizio De Santis2018-01-091-15/+8
| |/ / /
* | | | Log and send a system hook if a blocked user fails to loginsh-log-when-user-blockedStan Hu2018-01-141-1/+4
| | | | | | | | | | | | | | | | Closes #41633
* | | | Adds Rubocop rule for line break around conditionals🙈 jacopo beschi 🙉2018-01-111-0/+1
| | | |
* | | | Merge branch 'remove-soft-removals' into 'master'Douwe Maan2018-01-112-3/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove soft removals related code Closes #37447 See merge request gitlab-org/gitlab-ce!15789
| * | | | Remove soft removals related codeYorick Peterse2018-01-082-3/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes all usage of soft removals except for the "pending delete" system implemented for projects. This in turn simplifies all the query plans of the models that used soft removals. Since we don't really use soft removals for anything useful there's no point in keeping it around. This _does_ mean that hard removals of issues (which only admins can do if I'm not mistaken) can influence the "iid" values, but that code is broken to begin with. More on this (and how to fix it) can be found in https://gitlab.com/gitlab-org/gitlab-ce/issues/31114. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/37447
* | | | Merge branch 'Ben305/gitlab-ce-36669-default-mr-title-with-external-issues' ↵Grzegorz Bizon2018-01-111-6/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Resolve "Incorrect default merge request title when Jira activated" Closes #36669 See merge request gitlab-org/gitlab-ce!16356
| * | | | Never set special MR titles for external issuesSean McGivern2018-01-101-6/+2
| | | | |
| * | | | Fix incorrect default merge request title when external issue tracker is ↵Benedikt Huss2018-01-091-1/+1
| |/ / / | | | | | | | | | | | | activated
* | | | Merge branch '4020-rebase-message' into 'master'Sean McGivern2018-01-111-3/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve 'Rebase fails with "something went wrong" and huge ugly conflict error.' Closes #41820 See merge request gitlab-org/gitlab-ce!16326
| * | | | Store only generic message if rebase fails4020-rebase-messageJan Provaznik2018-01-091-3/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of storing detailed rebase error, only a generic message is stored with MR. The reason is that this message is exposed and displayed to end user and there is no reason to expose detailed backend information. Error message is still logged so detailed information can be found in logfile by admin if needed. Related #41820
* | | | Merge branch 'gcp-fix' into 'master'Kamil Trzciński2018-01-111-1/+4
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Fix GCP redirect Closes #41867 See merge request gitlab-org/gitlab-ce!16355
| * | | Fix billing checkingKamil Trzciński2018-01-101-1/+4
| |/ /
* | | Fix bug where award emojis would be lost when moving issues between projectssh-fix-award-emoji-move-issuesStan Hu2018-01-101-5/+11
|/ / | | | | | | Closes #33423
* | Merge branch '3968-protected-branch-is-not-set-for-default-branch-on-import' ↵Douwe Maan2018-01-082-20/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Protected branch is now created for default branch on import Closes #3968 See merge request gitlab-org/gitlab-ce!16198
| * | Protected branch is now created for default branch on import3968-protected-branch-is-not-set-for-default-branch-on-importTiago Botelho2018-01-062-20/+3
| | |
* | | Merge branch 'master' into ↵Matija Čupić2018-01-062-11/+9
|\ \ \ | | | | | | | | | | | | 39957-redirect-to-gpc-page-if-users-try-to-create-a-cluster-but-the-account-is-not-enabled
| * \ \ Merge branch '41249-clearing-the-cache' into 'master'Kamil Trzciński2018-01-061-0/+5
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Clearing the cache" Closes #41249 See merge request gitlab-org/gitlab-ce!16067
| | * \ \ Merge branch 'master' into 41249-clearing-the-cacheMatija Čupić2018-01-052-0/+54
| | |\ \ \
| | * \ \ \ Merge branch 'master' into 41249-clearing-the-cacheMatija Čupić2018-01-0412-18/+93
| | |\ \ \ \ | | | | |/ / | | | |/| |
| | * | | | Rename Project#cache_index to jobs_cache_indexMatija Čupić2018-01-041-1/+1
| | | | | |
| | * | | | Implement ResetProjectCacheServiceMatija Čupić2017-12-221-0/+1
| | | | | |
| | * | | | Merge branch 'master' into 41249-clearing-the-cacheMatija Čupić2017-12-227-22/+44
| | |\ \ \ \
| | * | | | | Stub ResetProjectCacheServiceMatija Čupić2017-12-211-0/+4
| | | | | | |
| * | | | | | Move git operations for multi_action into Gitlab::GitAlejandro Rodríguez2018-01-051-11/+4
| | |_|_|/ / | |/| | | |
* | | | | | Merge branch 'master' into ↵Matija Čupić2018-01-052-0/+54
|\ \ \ \ \ \ | |/ / / / / | | | | | | | | | | | | 39957-redirect-to-gpc-page-if-users-try-to-create-a-cluster-but-the-account-is-not-enabled
| * | | | | Backport 'Rebase' feature from EE to CEjprovazn-rebaseJan Provaznik2018-01-052-0/+54
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a project uses fast-forward merging strategy user has to rebase MRs to target branch before it can be merged. Now user can do rebase in UI by clicking 'Rebase' button instead of doing rebase locally. This feature was already present in EE, this is only backport of the feature to CE. Couple of changes: * removed rebase license check * renamed migration (changed timestamp) Closes #40301
* | | | | Return list of billing enabled projectsMatija Čupić2018-01-051-1/+1
| | | | |
* | | | | Merge branch 'master' into ↵Matija Čupić2018-01-0515-20/+99
|\ \ \ \ \ | |/ / / / | | | | | | | | | | 39957-redirect-to-gpc-page-if-users-try-to-create-a-cluster-but-the-account-is-not-enabled
| * | | | Merge branch 'osw-introduce-merge-request-statistics' into 'master'Sean McGivern2018-01-046-4/+58
| |\ \ \ \ | | |_|_|/ | |/| | | | | | | | | | | | | Improve closed/merged events queries performance on Projects::MergeRequestsController#show.json See merge request gitlab-org/gitlab-ce!15642
| | * | | Cache merged and closed events data in merge_request_metrics tableOswaldo Ferreira2018-01-026-4/+58
| | | | |
| * | | | Use if instead of unless on Projects::HashedStorage::MigrateRepositoryServiceda-handle-hashed-storage-repos-using-repo-import-taskDouglas Barbosa Alexandre2018-01-031-3/+3
| | | | |
| * | | | Fix Projects::TransferService#write_repository_config methodDouglas Barbosa Alexandre2018-01-031-1/+1
| | | | |
View Lorry Controller Status