Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix 500 error when loading an invalid upload URL | Sean McGivern | 2018-02-22 | 2 | -0/+10 |
| | |||||
* | Merge branch '14256-upload-destroy-removes-file' into 'master' | Sean McGivern | 2018-02-06 | 1 | -0/+6 |
|\ | | | | | | | | | | | | | Uploads should delete files when destroyed Closes #14256 See merge request gitlab-org/gitlab-ce!16799 | ||||
| * | remove file after `Upload#destroy` | Micaël Bergeron | 2018-02-06 | 1 | -0/+6 |
| | | | | | | | | | | it will also automatically prune empty directories for `FileUploader`-based uploaders. | ||||
* | | Drop filename enforcement | Shinya Maeda | 2018-02-06 | 1 | -6/+0 |
| | | |||||
* | | Add essential tests | Shinya Maeda | 2018-02-06 | 1 | -1/+7 |
| | | |||||
* | | Add CreateArtifactsTraceWorker | Shinya Maeda | 2018-02-06 | 1 | -5/+3 |
| | | |||||
* | | Expose current_path | Shinya Maeda | 2018-02-06 | 1 | -3/+5 |
| | | |||||
* | | JobArtifactUploader#open raise execption if its not Filestorage | Shinya Maeda | 2018-02-06 | 1 | -1/+3 |
| | | |||||
* | | Trace as artifacts | Shinya Maeda | 2018-02-06 | 1 | -0/+4 |
|/ | |||||
* | apply fixes from feedback42547-upload-store-mount-point | Micaël Bergeron | 2018-02-02 | 1 | -7/+9 |
| | |||||
* | add the uploader context to the upload model | Micaël Bergeron | 2018-02-02 | 4 | -15/+34 |
| | |||||
* | porting changes from upstream3867-port-to-ce | Micaël Bergeron | 2018-02-01 | 1 | -2/+2 |
| | |||||
* | porting upstream changes | Micaël Bergeron | 2018-02-01 | 1 | -2/+2 |
| | |||||
* | port of 594e6a0a625^..f74c90f68c6 | Micaël Bergeron | 2018-02-01 | 13 | -192/+250 |
| | |||||
* | Support uploads for groups | Jarka Kadlecova | 2017-12-07 | 2 | -4/+19 |
| | |||||
* | Fix specs failures, and use factory with `:ci_job_artifact, :archive` | Kamil Trzcinski | 2017-12-03 | 2 | -3/+19 |
| | |||||
* | Rename artifacts_* to legacy_artifacts_* | Kamil Trzcinski | 2017-12-03 | 3 | -52/+38 |
| | |||||
* | Fix legacy migration test | Zeger-Jan van de Weg | 2017-12-03 | 1 | -3/+1 |
| | |||||
* | JobArtifactsUploader does not inherrit from ArtifactsUploader | Zeger-Jan van de Weg | 2017-12-03 | 1 | -6/+20 |
| | |||||
* | Fix most test failures | Zeger-Jan van de Weg | 2017-12-03 | 1 | -8/+4 |
| | |||||
* | Fix creation of job_artifact_uploader | Kamil Trzcinski | 2017-12-03 | 1 | -6/+0 |
| | |||||
* | Rename Artifact to JobArtifact, split metadata out | Zeger-Jan van de Weg | 2017-12-03 | 2 | -5/+37 |
| | | | | | | | | | Two things at ones, as there was no clean way to seperate the commit and give me feedback from the tests. But the model Artifact is now JobArtifact, and the table does not have a type anymore, but the metadata is now its own model: Ci::JobArtifactMetadata. | ||||
* | Create Ci::Artifacts | Zeger-Jan van de Weg | 2017-12-03 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | To allow jobs/builds to have multiple artifacts, and to start seperating concerns from Ci::Build a new model is created: Ci::Artifact. Changes include the updating of the ArtifactUploader to adapt to a slightly different interface. The uploader expects to be initialized with a `Ci::Build`. Futher a migration with the minimal fields, the needed foreign keys and an index. Last, the way this works is by prepending a module to Ci::Build so we can basically override behaviour but if needed use `super` to get the original behaviour. | ||||
* | Changes to Attachments Migration for EE and Geo compatibility | Gabriel Mazetto | 2017-11-28 | 1 | -2/+9 |
| | |||||
* | FileUploader should check for hashed_storage?(:attachments) to use disk_path | Gabriel Mazetto | 2017-11-21 | 1 | -3/+7 |
| | |||||
* | Use the Hashed Storage compatible layer to store Attachments | Gabriel Mazetto | 2017-10-30 | 1 | -1/+1 |
| | |||||
* | Do not perform hard check (presence of file on storage) | Kamil Trzcinski | 2017-09-19 | 2 | -2/+2 |
| | | | | Instead perform soft check (the column set to indicate that file does exist) | ||||
* | Make sure uploads for personal snippets are correctly rendered | Bob Van Landuyt | 2017-08-11 | 1 | -1/+1 |
| | |||||
* | Rename more path_with_namespace -> full_path or disk_path | Gabriel Mazetto | 2017-08-01 | 1 | -1/+1 |
| | |||||
* | Merge branch '33359-pers-snippet-files-location' into 'security-9-3' | Sean McGivern | 2017-07-19 | 1 | -0/+4 |
| | | | | | Use uploads/system directory for personal snippets See merge request !2123 | ||||
* | Move system-uploads to `-/system` | Bob Van Landuyt | 2017-07-18 | 1 | -1/+1 |
| | |||||
* | Fix filename method of GitlabUploader to return always real filenamefix-filename-of-artifact-uploader | Kamil Trzcinski | 2017-06-13 | 1 | -0/+4 |
| | |||||
* | Merge branch 'sh-fix-refactor-uploader-work-dir' into 'master' | Kamil Trzciński | 2017-06-12 | 3 | -12/+23 |
|\ | | | | | | | | | | | | | Set artifact working directory to be in the destination store to prevent unnecessary I/O Closes #33274 See merge request !11905 | ||||
| * | Set artifact working directory to be in the destination store to prevent ↵sh-fix-refactor-uploader-work-dir | Stan Hu | 2017-06-06 | 3 | -12/+23 |
| | | | | | | | | | | | | | | | | | | | | | | | | unnecessary I/O Similar to #33218, build artifacts were being uploaded into a CarrierWave temporary directory in the Rails root directory before moved to their final destination, which could cause a copy across filesystems. This merge request refactors the work in !11866 so that any uploader can just override `work_dir` to change the default implementation. Closes #33274 | ||||
* | | Make the uploader use the updated folder | Bob Van Landuyt | 2017-06-08 | 1 | -1/+3 |
| | | |||||
* | | Bring in security changes from the 9.2.5 release | DJ Mountney | 2017-06-08 | 2 | -4/+21 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Ran: - git format-patch v9.2.2..v9.2.5 --stdout > patchfile.patch - git checkout -b 9-2-5-security-patch origin/v9.2.2 - git apply patchfile.patch - git commit - [Got the sha ref for the commit] - git checkout -b upstream-9-2-security master - git cherry-pick <SHA of the patchfile commit> - [Resolved conflicts] - git cherry-pick --continue | ||||
* | | Merge branch '12910-snippets-description' into 'master' | Phil Hughes | 2017-06-07 | 3 | -4/+72 |
|\ \ | |/ |/| | | | | | | | | | Support descriptions for snippets Closes #31894 See merge request !11071 | ||||
| * | Support uploads for newly created personal snippets12910-snippets-description | Jarka Kadlecova | 2017-06-07 | 2 | -10/+26 |
| | | |||||
| * | Support descriptions for snippets | Jarka Kadlecova | 2017-05-31 | 2 | -1/+53 |
| | | |||||
* | | Fix LFS timeouts when trying to save large filessh-fix-lfs-from-moving-across-filesystems | Stan Hu | 2017-06-04 | 1 | -0/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following was happening: 1. Workhorse stores an LFS file in /var/opt/gitlab/gitlab-rails/shared/lfs-objects 2. CarrierWave then renames the file to a temporary directory (e.g. /opt/gitlab/embedded/service/gitlab-rails/tmp) 3. CarrierWave then renames the file to its final location (e.g. /var/opt/gitlab/gitlab-rails/shared/lfs-objects) When the LFS upload path was on a different filesystem than the Rails installation, step 2 could take a longer than 10 seconds, at which point Workhorse would time out with "badgateway: failed after 10s: context canceled". This change makes the work path in the same root as the LFS storage path, preventing moves across filesystems. Closes #33218 | ||||
* | | Add missing specs | Kamil Trzcinski | 2017-06-01 | 3 | -8/+8 |
| | | |||||
* | | Fix data inconsistency issue for old artifacts by moving them to a currently ↵ | Kamil Trzcinski | 2017-06-01 | 3 | -19/+25 |
|/ | | | | used path | ||||
* | Support uploaders for personal snippets comments12910-uploader-pers-snippet | Jarka Kadlecova | 2017-05-02 | 5 | -15/+22 |
| | |||||
* | Periodically clean up temporary upload files to recover storage space | blackst0ne | 2017-04-10 | 1 | -4/+0 |
| | |||||
* | Handle relative and absolute Upload paths in the Uploaders | Robert Speicher | 2017-03-06 | 3 | -6/+45 |
| | |||||
* | Add `RecordsUploads` module to record Upload records via callbacks | Robert Speicher | 2017-03-06 | 4 | -0/+46 |
| | |||||
* | Minor refactoring of Uploaders | Robert Speicher | 2017-02-24 | 6 | -23/+26 |
| | | | | | | | | | - Moves a duplicate `file_storage?` definition into the common `GitlabUploader` ancestor. - Get the `uploads` base directory from a class method rather than hard-coding it where it's needed. This will be used in a subsequent MR to store Uploads in the database. - Improves the specs for uploaders. | ||||
* | Enable Style/MutableConstant | Douwe Maan | 2017-02-23 | 1 | -3/+3 |
| | |||||
* | Merge branch 'svg-xss-fix' into 'security' | Robert Speicher | 2017-02-15 | 2 | -2/+9 |
| | | | | | Fix for XSS vulnerability in SVG attachments See https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2059 | ||||
* | Copy, don't move uploaded avatar files | Jacob Vosmaer | 2017-01-03 | 1 | -0/+11 |
| |