gitlab-org--gitlab-foss/spec/uploaders
Stan Hu a1f44c1b49 Fix incorrect prefix used in new uploads for personal snippets
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/24550 fixed the
case where the wrong path on disk was being searched, but it
inadvertently ommitted the `/uploads/-/system` prefix when rendering the
Markdown for personal snippet uploads when they were stored directly in
object storage.

A personal snippet path is stored using FileUploader#upload_path.
The format for the path:

Local storage: :random_hex/:filename.
Object storage: personal_snippet/:id/:random_hex/:filename.

upload_paths represent the possible paths for a given identifier,
which will vary depending on whether the file is stored in local or
object storage. upload_path should match an element in upload_paths.

base_dir represents the path seen by the user in Markdown, and it
should always be prefixed with uploads/-/system.

store_dirs represent the paths that are actually used on disk. For
object storage, this should omit the prefix /uploads/-/system.

For example, consider the requested path
/uploads/-/system/personal_snippet/172/ff4ad5c2/file.png.

For local storage:

base_dir: uploads/-/system/personal_snippet/172
upload_path: ff4ad5c2/file.png
upload_paths: ["ff4ad5c2/file.png", "personal_snippet/172/ff4ad5c2/file.png"].
store_dirs: {1=>"uploads/-/system/personal_snippet/172/ff4ad5c2",
             2=>"personal_snippet/172/ff4ad5c2"}

For object storage:

upload_path: personal_snippet/172/ff4ad5c2/file.png

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61671
2019-05-16 23:36:55 -07:00
..
workers/object_storage apply feedback 2018-06-12 11:22:35 -04:00
attachment_uploader_spec.rb Make all uses of fixture_file_upload use relative paths 2018-06-07 09:54:41 -05:00
avatar_uploader_spec.rb fix avatar uploader error 2018-09-10 17:41:51 +02:00
external_diff_uploader_spec.rb Allow MR diffs to be placed into an object store 2019-02-05 14:12:48 +00:00
file_mover_spec.rb Check snippet attached file to be moved is within designated directory 2019-02-21 16:44:44 +08:00
file_uploader_spec.rb Enable the Layout/ExtraSpacing cop 2019-01-24 13:05:45 +01:00
gitlab_uploader_spec.rb Fix and add specs for testing metadata entry 2018-07-09 14:19:52 +02:00
import_export_uploader_spec.rb Clean up CarrierWave's import/export files 2019-05-07 01:27:50 -07:00
job_artifact_uploader_spec.rb Squashed commit of the following: 2018-08-27 09:12:02 +02:00
legacy_artifact_uploader_spec.rb Correct more usages of non-relative fixture_file_upload paths 2018-06-07 09:54:41 -05:00
lfs_object_uploader_spec.rb Fix file_store for artifacts and lfs when saving 2018-05-01 22:27:54 +02:00
namespace_file_uploader_spec.rb Fix already initialized constant IDENTIFIER warning 2018-12-10 11:00:08 +05:00
object_storage_spec.rb Optimise upload path calls 2019-05-14 11:01:48 -05:00
personal_file_uploader_spec.rb Fix incorrect prefix used in new uploads for personal snippets 2019-05-16 23:36:55 -07:00
records_uploads_spec.rb Autocorrect with RSpec/ExampleWording cop 2019-04-05 08:43:27 +00:00
uploader_helper_spec.rb Uploads to wiki stored inside the wiki git repository 2018-09-04 10:39:08 +00:00