summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Use factory instead of using new directly. Feedback:show-all-pipelines-from-all-diffsLin Jen-Shin2016-09-211-9/+2
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15765156
* Add a test for non-persisted merge requestLin Jen-Shin2016-09-201-0/+22
|
* If merge request wasn't persisted yet, we show only 1 pipeline:Lin Jen-Shin2016-09-201-7/+11
| | | | | | | | | | | | However, if MergeRequest#all_commits_sha would want to handle non-persisted merge request, by judging its name, it should not just give 1 SHA, but all of them. But we don't really care all_commits_sha for non-persisted merge request anyway. So I think we should just ignore that case. Better to not implementing something than implementing it in a wrong and confusing way.
* Test against MergeRequest#all_commits_sha, feedback:Lin Jen-Shin2016-09-201-9/+22
| | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15750344 So that we could just use it in testing for MergeRequest#all_pipelines
* slightly tweak about the comment, feedback:Lin Jen-Shin2016-09-201-1/+1
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15750250
* Introduce MergeRequest#all_commits_sha, feedback:Lin Jen-Shin2016-09-201-8/+9
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15746083
* Add an entry to CHANGELOG [ci skip]Lin Jen-Shin2016-09-201-0/+1
|
* Show all pipelines from all merge_request_diffs:Lin Jen-Shin2016-09-193-9/+44
| | | | | This way we could also show pipelines from commits which were discarded due to a force push.
* Merge branch 'per-build-token-without-lfs' into 'master' Rémy Coutable2016-09-1930-201/+985
|\ | | | | | | | | | | | | | | | | Make CI to use the permission of the user who is trigger the build This is continuation of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5735, but with removed all LFS code that is added by: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6043. This also incorporates most of LFS code added in !6043 to simplify further merge. See merge request !6409
| * Solve code review commentsKamil Trzcinski2016-09-192-1/+10
| |
| * Properly support Gitlab::Auth::ResultKamil Trzcinski2016-09-192-8/+8
| |
| * Revert all changes introduced by ↵Kamil Trzcinski2016-09-1913-245/+3
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6043
| * Move Gitlab::Auth.Result to separate fileKamil Trzcinski2016-09-192-14/+17
| |
| * Added CHANGELOGKamil Trzcinski2016-09-191-0/+1
| |
| * Merge branch 'master' into per-build-tokenKamil Trzcinski2016-09-19150-1176/+1674
| |\
| * | Fix spec failuresKamil Trzcinski2016-09-192-2/+2
| | |
| * | Support pushing via SSHKamil Trzcinski2016-09-192-1/+12
| | |
| * | Don't leak build tokens in build logsKamil Trzcinski2016-09-195-15/+113
| | |
| * | Fix permissions for creating container imagesKamil Trzcinski2016-09-191-1/+1
| | |
| * | Improve authentication_result usageKamil Trzcinski2016-09-162-13/+20
| | |
| * | Fix specs for available statusesKamil Trzcinski2016-09-161-2/+2
| | |
| * | Simplify checking of allowed abilities in git_http_client_controllerKamil Trzcinski2016-09-164-59/+60
| | |
| * | Improve code commentsKamil Trzcinski2016-09-163-4/+4
| | |
| * | Merge remote-tracking branch 'origin/lfs-support-for-ssh' into per-build-tokenKamil Trzcinski2016-09-160-0/+0
| |\ \
| | * | Refactored authentication code to make it a bit clearer, added test for ↵lfs-support-for-sshPatricio Cano2016-09-156-38/+52
| | | | | | | | | | | | | | | | wrong SSH key.
| * | | Added missing LFS specsKamil Trzcinski2016-09-161-22/+213
| | | |
| * | | Fix specs after renaming authentication_capabilitiesKamil Trzcinski2016-09-163-14/+16
| | | |
| * | | Rename capabilities to authentication_abilitiesKamil Trzcinski2016-09-1611-62/+74
| | | |
| * | | Added builds_spec and git_http_specsKamil Trzcinski2016-09-154-20/+130
| | | |
| * | | Fix specs after merging LFS changesKamil Trzcinski2016-09-153-8/+36
| | | |
| * | | Merge remote-tracking branch 'origin/lfs-support-for-ssh' into per-build-tokenKamil Trzcinski2016-09-1532-112/+585
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # app/controllers/projects/git_http_client_controller.rb # app/helpers/lfs_helper.rb # lib/gitlab/auth.rb # spec/requests/lfs_http_spec.rb
| | * | | Refactored authentication code to make it a bit clearer, added test for ↵Patricio Cano2016-09-156-38/+50
| | |/ / | | | | | | | | | | | | wrong SSH key.
| | * | Further refactoring of authentication code, and code style fixes.Patricio Cano2016-09-153-47/+48
| | | |
| | * | Use special characters for `lfs+deploy-key` to prevent a someone from ↵Patricio Cano2016-09-157-9/+9
| | | | | | | | | | | | | | | | creating a user with this username, and method name refactoring.
| | * | Better authentication handling, syntax fixes and better actor handling for ↵Patricio Cano2016-09-156-45/+51
| | | | | | | | | | | | | | | | LFS Tokens
| | * | Improve string handling.Patricio Cano2016-09-151-2/+2
| | | |
| | * | Refactored handling of the `LfsToken` and added functionality to it to ↵Patricio Cano2016-09-155-21/+46
| | | | | | | | | | | | | | | | simplify external code.
| | * | Refactored LFS auth logic when using SSH to use its own API endpoint ↵Patricio Cano2016-09-157-32/+62
| | | | | | | | | | | | | | | | `/lfs_authenticate` and added tests.
| | * | Refactor LFS token logic to use a Redis key instead of a DB field, making it ↵Patricio Cano2016-09-1515-79/+93
| | | | | | | | | | | | | | | | a 1 use only token.
| | * | Added CHANGELOG item and documentation.Patricio Cano2016-09-153-2/+11
| | | |
| | * | Added LFS support to SSHPatricio Cano2016-09-1513-21/+169
| | | | | | | | | | | | | | | | - Required on the GitLab Rails side is mostly authentication and API related.
| * | | Verify permission of build in context of dependent projectKamil Trzcinski2016-09-152-8/+38
| | | |
| * | | Fix LFS specsKamil Trzcinski2016-09-151-4/+4
| | | |
| * | | Fix most of specsKamil Trzcinski2016-09-157-20/+39
| | | |
| * | | Simplify LFS helperKamil Trzcinski2016-09-151-4/+0
| | | |
| * | | Fix existing authorization specsKamil Trzcinski2016-09-1511-34/+51
| | | |
| * | | Merge remote-tracking branch 'origin/master' into per-build-tokenKamil Trzcinski2016-09-15104-502/+1289
| |\ \ \ | | | | | | | | | | | | | | | | | | | | # Conflicts: # db/schema.rb
| * | | | Add access specsKamil Trzcinski2016-09-155-45/+168
| | | | |
| * | | | Use `build_read_container_image` and use `build_download_code`Kamil Trzcinski2016-09-156-53/+50
| | | | |
| * | | | Refactor Gitlab::Auth to simplify the data flowKamil Trzcinski2016-09-141-34/+40
| | | | |