summaryrefslogtreecommitdiff
path: root/app/controllers/projects/artifacts_controller.rb
Commit message (Collapse)AuthorAgeFilesLines
* Add latest changes from gitlab-org/security/gitlab@14-8-stable-eeGitLab Bot2022-04-291-0/+7
|
* Add latest changes from gitlab-org/gitlab@14-1-stable-eev14.1.0-rc42GitLab Bot2021-07-201-1/+1
|
* Add latest changes from gitlab-org/gitlab@13-5-stable-eev13.5.0-rc42GitLab Bot2020-10-211-0/+2
|
* Add latest changes from gitlab-org/gitlab@13-3-stable-eeGitLab Bot2020-08-201-1/+1
|
* Add latest changes from gitlab-org/gitlab@13-0-stable-eeGitLab Bot2020-05-291-2/+10
|
* Add latest changes from gitlab-org/gitlab@13-0-stable-eeGitLab Bot2020-05-201-3/+12
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2019-09-181-1/+37
|
* Find build by sha from refMatija Čupić2019-07-231-1/+4
| | | | Adds ability to find builds by sha when only specifying a ref.
* Remove get_build method for find_by_idSteve Azzopardi2019-01-091-1/+1
| | | | | | The original intention of `get_build` was as a workaround not to violate `CodeReuse/ActiveRecord`. `find_by_id` does the exact same thing but does not violate the rubocop rule.
* Create `get_build` for project modelSteve Azzopardi2019-01-071-3/+1
| | | | | | | | Inside of `Projects::ArtifactsController` and `Projects::BuildArtifactsController` we fetching the build by id using active record directly which violates `CodeReuse/ActiveRecord` rubocop rule. Create `get_build` inside of `project` model which does the same thing.
* Refactor project.latest_successful_builds_for defSteve Azzopardi2019-01-071-4/+1
| | | | | | | | | | | | | | `project.latest_successful_builds_for(ref)` is being used to find a single job all the time. This results into us having to call `find_by` inside of the controller which violates our CodeReuse/ActiveRecord rubocop rule. Refactor `project.latest_successful_builds_for(ref)` to `project.latest_successful_build_for(job_name, ref)` which will execute the `find_by` inside of the model. Also create `project.latest_successful_build_for!(job_name, ref)` which raises an exception instead of returning nil.
* Revert "[Rails5] Set request.format for artifacts_controller"Douwe Maan2018-12-031-9/+0
| | | | | | | This reverts commit c48e7dc2633f8ee8f38150b050938f43e75bd207. # Conflicts: # app/controllers/projects/artifacts_controller.rb
* backport: Always proxy reports downloadsalways-proxy-reportsKamil Trzciński2018-11-071-1/+1
| | | | This makes to always proxy reports
* Disable artifact validation for download actionMatija Čupić2018-10-161-2/+2
|
* CE Resolve "Refactor code quality similar to JUnit tests"Matija Čupić2018-10-021-1/+3
|
* Enable even more frozen string in app/controllersgfyoung2018-09-251-0/+2
| | | | | | | | | | | | Enables frozen string for some vestigial files as well as the following: * app/controllers/projects/**/*.rb * app/controllers/sherlock/**/*.rb * app/controllers/snippets/**/*.rb * app/controllers/users/**/*.rb Partially addresses #47424.
* Disable existing offenses for the CodeReuse copsYorick Peterse2018-09-111-0/+4
| | | | | This whitelists all existing offenses for the various CodeReuse cops, of which most are triggered by the CodeReuse/ActiveRecord cop.
* [Rails5] Set request.format for artifacts_controllerblackst0ne-rails5-expected-the-response-to-have-status-code-ok-but-it-was-404blackst0ne2018-06-161-0/+9
| | | | | | | Since Rails 5 request.format gets set by extension if it's not explicitly set. That means artifacts_controller#file no longer works as expected. This commit explicitly sets request.format for artifacts_controller#file.
* Merge branch 'zj-multiple-artifacts-ee' into 'master'Grzegorz Bizon2018-02-281-3/+2
| | | | | | Multiple artifacts ee See merge request gitlab-org/gitlab-ee!3276
* Merge commit '11c67e7c2f992299ff5918ce67995b73d1e0be6d' into ↵Kamil Trzciński2018-02-281-7/+11
|\ | | | | | | object-storage-ee-to-ce-backport
| * Test against public projectsZeger-Jan van de Weg2017-10-051-1/+1
| |
| * Redirect to pages daemonZeger-Jan van de Weg2017-10-031-7/+11
| |
* | Merge branch 'jej/lfs-object-storage' into 'master'Douwe Maan2018-02-281-5/+2
|/ | | | | | | | Can migrate LFS objects to S3 style object storage Closes #2841 See merge request !2760
* Do not use artifacts metadata to access single artifactGrzegorz Bizon2017-09-051-5/+7
|
* Create and use project path helpers that only need a project, no namespaceDouwe Maan2017-07-051-1/+1
|
* Merge branch 'dm-collapsed-blob' into 'master'Sean McGivern2017-06-011-1/+1
|\ | | | | | | | | Consistent diff and blob size limit names See merge request !11776
| * Consistent diff and blob size limit namesDouwe Maan2017-05-291-1/+1
| |
* | Rename BuildsController to JobsControllerLin Jen-Shin2017-05-171-2/+2
|/ | | | | | | | | | | | Rename other URL generators admin_builds_path -> admin_jobs_path Fix tests and more renaming Fix more tests Also change build_id to job_id in the controller
* Add artifact file page that uses the blob viewerdm-artifact-blob-viewerDouwe Maan2017-05-041-5/+23
|
* Add breadcrumb, build header and pipelines submenu to artifacts browserDouwe Maan2017-05-031-2/+6
|
* Extract ref_name and path in before_action, feedback:Lin Jen-Shin2016-08-261-5/+10
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14469768
* Just use instance variable instead, feedback:Lin Jen-Shin2016-08-251-7/+2
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14400736
* Path could also have slashes! Feedback:Lin Jen-Shin2016-08-241-3/+11
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14347729
* Not sure why I missed this renamingLin Jen-Shin2016-08-181-1/+1
|
* Use path rather than URL because it should work for http 302:Lin Jen-Shin2016-08-181-2/+2
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14035941
* Use switch case in a helper, feedback:Lin Jen-Shin2016-08-171-7/+3
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13988401
* Rename to latest_succeeded, feedback:Lin Jen-Shin2016-08-161-1/+1
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13908017
* Use URL helper, feedback:Lin Jen-Shin2016-08-161-3/+7
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13880889
* Rename latest_success* to latest_successful:Lin Jen-Shin2016-07-191-1/+1
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13164642
* Merge branch 'master' into artifacts-from-ref-and-build-nameLin Jen-Shin2016-07-181-3/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (335 commits) Disable transaction when adding index for Ci::Pipeline added changelog allow empty repos on import/export Modify test for Build tabs Add tests to project builds for pending tab Remove unused .js-running-count class Add test for new pending tab and update tests for running tab Add Pending Tab to Admin Builds added changelog limit project expor retry to only 3 use method in validates statement Fix spec to set import_url before attempting to create import_data Allow a project import URL to be blank to prevent false positives preventing settings from being saved Refactor gitlab_ci_yaml_processor variables tests Fix CI yaml example Align cancel and retry buttons Remove deploy to production button Remove irrelevant comments Fix gitlab_ci_yaml_processor_spec.rb Fix AddWhenAndYamlVariablesToCiBuilds migration ...
| * Just give regular 404, feedback:Lin Jen-Shin2016-07-111-1/+1
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5094#note_12984211
| * Use Gitlab-Workhorse-Send-Data to send entry:Lin Jen-Shin2016-07-051-2/+1
| | | | | | | | | | | | | | Closes #19224, Closes #19128 Also requires this MR to work: https://gitlab.com/gitlab-org/gitlab-workhorse/merge_requests/53
* | Use find_by, feedback:Lin Jen-Shin2016-07-181-1/+1
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13125486
* | Introduce Project#latest_success_builds_for:Lin Jen-Shin2016-07-151-2/+2
| | | | | | | | | | So it's more accessible for views to access the names of jobs. Only filter Build#name from where we really need to download it.
* | Update routes based on feedback from:Lin Jen-Shin2016-07-141-1/+1
| | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13058785 And note that job/build_name could contain `/`
* | Rename to ref_name so it's aligning with APILin Jen-Shin2016-07-131-2/+2
| |
* | Could be faster when params[:path] is missingLin Jen-Shin2016-07-121-2/+2
| |
* | Add all the tests and fix stuffs along the way:Lin Jen-Shin2016-07-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | It turns out they are different: builds.success.latest.first and builds.latest.success.first If we put success first, that latest would also filter via success, and which is what we want here.
* | Remove redundant returnLin Jen-Shin2016-07-111-1/+1
| |
* | Using plain if/else is much easier to understandLin Jen-Shin2016-07-111-4/+4
| |
View Lorry Controller Status