Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Use factory instead of using new directly. Feedback:show-all-pipelines-from-all-diffs | Lin Jen-Shin | 2016-09-21 | 1 | -9/+2 |
| | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15765156 | ||||
* | Add a test for non-persisted merge request | Lin Jen-Shin | 2016-09-20 | 1 | -0/+22 |
| | |||||
* | If merge request wasn't persisted yet, we show only 1 pipeline: | Lin Jen-Shin | 2016-09-20 | 1 | -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-Shin | 2016-09-20 | 1 | -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-Shin | 2016-09-20 | 1 | -1/+1 |
| | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15750250 | ||||
* | Introduce MergeRequest#all_commits_sha, feedback: | Lin Jen-Shin | 2016-09-20 | 1 | -8/+9 |
| | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6414#note_15746083 | ||||
* | Add an entry to CHANGELOG [ci skip] | Lin Jen-Shin | 2016-09-20 | 1 | -0/+1 |
| | |||||
* | Show all pipelines from all merge_request_diffs: | Lin Jen-Shin | 2016-09-19 | 3 | -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 Coutable | 2016-09-19 | 30 | -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 comments | Kamil Trzcinski | 2016-09-19 | 2 | -1/+10 |
| | | |||||
| * | Properly support Gitlab::Auth::Result | Kamil Trzcinski | 2016-09-19 | 2 | -8/+8 |
| | | |||||
| * | Revert all changes introduced by ↵ | Kamil Trzcinski | 2016-09-19 | 13 | -245/+3 |
| | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6043 | ||||
| * | Move Gitlab::Auth.Result to separate file | Kamil Trzcinski | 2016-09-19 | 2 | -14/+17 |
| | | |||||
| * | Added CHANGELOG | Kamil Trzcinski | 2016-09-19 | 1 | -0/+1 |
| | | |||||
| * | Merge branch 'master' into per-build-token | Kamil Trzcinski | 2016-09-19 | 150 | -1176/+1674 |
| |\ | |||||
| * | | Fix spec failures | Kamil Trzcinski | 2016-09-19 | 2 | -2/+2 |
| | | | |||||
| * | | Support pushing via SSH | Kamil Trzcinski | 2016-09-19 | 2 | -1/+12 |
| | | | |||||
| * | | Don't leak build tokens in build logs | Kamil Trzcinski | 2016-09-19 | 5 | -15/+113 |
| | | | |||||
| * | | Fix permissions for creating container images | Kamil Trzcinski | 2016-09-19 | 1 | -1/+1 |
| | | | |||||
| * | | Improve authentication_result usage | Kamil Trzcinski | 2016-09-16 | 2 | -13/+20 |
| | | | |||||
| * | | Fix specs for available statuses | Kamil Trzcinski | 2016-09-16 | 1 | -2/+2 |
| | | | |||||
| * | | Simplify checking of allowed abilities in git_http_client_controller | Kamil Trzcinski | 2016-09-16 | 4 | -59/+60 |
| | | | |||||
| * | | Improve code comments | Kamil Trzcinski | 2016-09-16 | 3 | -4/+4 |
| | | | |||||
| * | | Merge remote-tracking branch 'origin/lfs-support-for-ssh' into per-build-token | Kamil Trzcinski | 2016-09-16 | 0 | -0/+0 |
| |\ \ | |||||
| | * | | Refactored authentication code to make it a bit clearer, added test for ↵lfs-support-for-ssh | Patricio Cano | 2016-09-15 | 6 | -38/+52 |
| | | | | | | | | | | | | | | | | wrong SSH key. | ||||
| * | | | Added missing LFS specs | Kamil Trzcinski | 2016-09-16 | 1 | -22/+213 |
| | | | | |||||
| * | | | Fix specs after renaming authentication_capabilities | Kamil Trzcinski | 2016-09-16 | 3 | -14/+16 |
| | | | | |||||
| * | | | Rename capabilities to authentication_abilities | Kamil Trzcinski | 2016-09-16 | 11 | -62/+74 |
| | | | | |||||
| * | | | Added builds_spec and git_http_specs | Kamil Trzcinski | 2016-09-15 | 4 | -20/+130 |
| | | | | |||||
| * | | | Fix specs after merging LFS changes | Kamil Trzcinski | 2016-09-15 | 3 | -8/+36 |
| | | | | |||||
| * | | | Merge remote-tracking branch 'origin/lfs-support-for-ssh' into per-build-token | Kamil Trzcinski | 2016-09-15 | 32 | -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 Cano | 2016-09-15 | 6 | -38/+50 |
| | |/ / | | | | | | | | | | | | | wrong SSH key. | ||||
| | * | | Further refactoring of authentication code, and code style fixes. | Patricio Cano | 2016-09-15 | 3 | -47/+48 |
| | | | | |||||
| | * | | Use special characters for `lfs+deploy-key` to prevent a someone from ↵ | Patricio Cano | 2016-09-15 | 7 | -9/+9 |
| | | | | | | | | | | | | | | | | creating a user with this username, and method name refactoring. | ||||
| | * | | Better authentication handling, syntax fixes and better actor handling for ↵ | Patricio Cano | 2016-09-15 | 6 | -45/+51 |
| | | | | | | | | | | | | | | | | LFS Tokens | ||||
| | * | | Improve string handling. | Patricio Cano | 2016-09-15 | 1 | -2/+2 |
| | | | | |||||
| | * | | Refactored handling of the `LfsToken` and added functionality to it to ↵ | Patricio Cano | 2016-09-15 | 5 | -21/+46 |
| | | | | | | | | | | | | | | | | simplify external code. | ||||
| | * | | Refactored LFS auth logic when using SSH to use its own API endpoint ↵ | Patricio Cano | 2016-09-15 | 7 | -32/+62 |
| | | | | | | | | | | | | | | | | `/lfs_authenticate` and added tests. | ||||
| | * | | Refactor LFS token logic to use a Redis key instead of a DB field, making it ↵ | Patricio Cano | 2016-09-15 | 15 | -79/+93 |
| | | | | | | | | | | | | | | | | a 1 use only token. | ||||
| | * | | Added CHANGELOG item and documentation. | Patricio Cano | 2016-09-15 | 3 | -2/+11 |
| | | | | |||||
| | * | | Added LFS support to SSH | Patricio Cano | 2016-09-15 | 13 | -21/+169 |
| | | | | | | | | | | | | | | | | - Required on the GitLab Rails side is mostly authentication and API related. | ||||
| * | | | Verify permission of build in context of dependent project | Kamil Trzcinski | 2016-09-15 | 2 | -8/+38 |
| | | | | |||||
| * | | | Fix LFS specs | Kamil Trzcinski | 2016-09-15 | 1 | -4/+4 |
| | | | | |||||
| * | | | Fix most of specs | Kamil Trzcinski | 2016-09-15 | 7 | -20/+39 |
| | | | | |||||
| * | | | Simplify LFS helper | Kamil Trzcinski | 2016-09-15 | 1 | -4/+0 |
| | | | | |||||
| * | | | Fix existing authorization specs | Kamil Trzcinski | 2016-09-15 | 11 | -34/+51 |
| | | | | |||||
| * | | | Merge remote-tracking branch 'origin/master' into per-build-token | Kamil Trzcinski | 2016-09-15 | 104 | -502/+1289 |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | # Conflicts: # db/schema.rb | ||||
| * | | | | Add access specs | Kamil Trzcinski | 2016-09-15 | 5 | -45/+168 |
| | | | | | |||||
| * | | | | Use `build_read_container_image` and use `build_download_code` | Kamil Trzcinski | 2016-09-15 | 6 | -53/+50 |
| | | | | | |||||
| * | | | | Refactor Gitlab::Auth to simplify the data flow | Kamil Trzcinski | 2016-09-14 | 1 | -34/+40 |
| | | | | |