aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | Accept S3 upload options (e.g. server_side_encryption)James Baer2017-07-202-3/+23
| |
* | Use descriptive new freeze_time helperGeorge Claghorn2017-07-194-4/+4
| |
* | Replace hard-coded S3 bucket name with configured bucketJames T. Perreault2017-07-172-2/+2
| |
* | Remove unused requireGeorge Claghorn2017-07-171-1/+0
| |
* | Add direct upload support to GCS serviceMichael Herold2017-07-173-2/+63
| |
* | Splat optionsGeorge Claghorn2017-07-161-16/+2
| |
* | Fix S3 direct upload testGeorge Claghorn2017-07-161-13/+3
| |
* | Merge pull request #56 from colorfulfool/fix-s3-serviceGeorge Claghorn2017-07-151-3/+1
|\ \ | | | | | | Fix a typo in S3Service
| * | Fix a typo in S3Servicecolorfulfool2017-07-161-3/+1
|/ /
* | Swap branchesGeorge Claghorn2017-07-151-3/+3
| |
* | Merge pull request #48 from seanhandley/hosted_s3George Claghorn2017-07-151-2/+18
|\ \ | | | | | | Allow custom endpoints for S3.
| * | Allow custom endpoints for S3.Sean Handley2017-07-151-2/+18
| | |
* | | Put conditional backGeorge Claghorn2017-07-141-2/+2
| | |
* | | Merge pull request #55 from SGospodinov/logger-for-5.2.0George Claghorn2017-07-141-3/+3
|\ \ \ | | | | | | | | Fixing logger to work with Rails 5.2.0.alpha
| * | | Fixing logger to work with Rails 5.2.0.alphaStanislav Gospodinov2017-07-151-3/+3
|/ / /
* | | Depend on Rails >= 5.2.0.alphaGeorge Claghorn2017-07-143-35/+51
| | |
* | | Merge pull request #53 from colorfulfool/error-messagesGeorge Claghorn2017-07-142-2/+5
|\ \ \ | | | | | | | | Useful error messages for two possible misconfigurations
| * | | Fail early if no storage service is specifiedcolorfulfool2017-07-151-0/+3
| | | |
| * | | Don't fail on boot because of missing S3 keys when S3 is not usedcolorfulfool2017-07-151-2/+2
| |/ /
* | | Merge pull request #51 from dixpac/restore_intermediate_variablesGeorge Claghorn2017-07-141-2/+8
|\ \ \ | |/ / |/| | Revert back to the original implementaion with varaibles
| * | Revert back to the original implementaion with varaiblesdixpac2017-07-141-2/+8
|/ / | | | | | | | | | | Revert `exist? and url` to the original implementation. Since the new one doesn't provide any benefits and makes implementation harder to follow.
* | Added rubocop / codeclimate config and fixed current offenses (#45)Cristian Bica2017-07-1418-33/+183
| |
* | Fix regular expression on s3 test URL generation test (#44)Dino Maric2017-07-131-5/+5
| | | | | | | | | | | | | | So tests are passing if the bucket name is rails-active storage. But developers specify their own s3 tests configuration (in my case was activestorage-test) then this regex fails. Also the first part is dynamic and based on bucket name and region
* | Merge pull request #43 from dixpac/small_gcs_service_refactoringGeorge Claghorn2017-07-131-8/+2
|\ \ | | | | | | Remove unnecessary variables from gcs_service implementation
| * | Remove few ivars from gcs_service implementationdixpac2017-07-131-8/+2
|/ /
* | Merge pull request #42 from quartzmo/gcs-site-urlGeorge Claghorn2017-07-123-6/+6
|\ \ | | | | | | Update GCSService#url
| * | Update GCSService#urlChris Smith2017-07-123-6/+6
| | | | | | | | | | | | | | | Update google-cloud-storage dependency to 1.3 Refactor arguments to Google::Cloud::Storage::File#signed_url
* | | Revert "S3: slim down service implementation (#40)" (#41)David Heinemeier Hansson2017-07-121-24/+36
|/ / | | | | This reverts commit 6d3962461fb8d35fc9538d685fee96267663acf2.
* | S3: slim down service implementation (#40)Jeremy Daer2017-07-111-36/+24
| | | | | | | | | | | | | | * Use simple core API for duck-type compat with other clients * initialize: accept an existing client * initialize: accept arbitrary client args instead of a fixed, required set * download: use native get_object streaming, no need to implement range requests * exists?: use head_object (which returns immediately) rather than waiting for existence
* | Revert "Depend on Rails > 5.1"George Claghorn2017-07-111-4/+4
| | | | | | | | This reverts commit 53d5384ac265ead44b25eec5f8f5020568184da2.
* | Depend on Rails > 5.1George Claghorn2017-07-111-4/+4
| |
* | Expose chunked downloadsGeorge Claghorn2017-07-103-6/+19
| |
| * Very incomplete first stabDavid Heinemeier Hansson2017-07-1110-24/+134
|/
* Scope aws-skd to version 2 (#34)Dino Maric2017-07-102-2/+2
| | | | | Since we use new aws-sdk API, I've scoped aws-sdk version so someone doesn't accidentaly installs wrong version during the development.
* Configurator tests: work against test-local configJeremy Daer2017-07-091-3/+3
| | | | | | So tests pass when service configs aren't set up. References #28
* Tests: Dir.mktmpdir neatly wraps up tmpdir + joinJeremy Daer2017-07-093-6/+5
|
* Add basic tests to the Configurator#build (#28)Dino Maric2017-07-091-0/+15
|
* Accept sgids for existing blobs created via direct upload as an attachableDavid Heinemeier Hansson2017-07-092-0/+9
|
* Direct uploads for S3David Heinemeier Hansson2017-07-0916-21/+138
|
* Copypasta commentsDavid Heinemeier Hansson2017-07-091-3/+0
| | | | | | | | | | # Conflicts: # lib/active_storage/engine.rb # lib/active_storage/service.rb # lib/active_storage/service/disk_service.rb # lib/active_storage/service/s3_service.rb # test/service/s3_service_test.rb # test/test_helper.rb
* Proper logging is now in placeDavid Heinemeier Hansson2017-07-091-1/+0
|
* Test URL generation for S3 and DiskDavid Heinemeier Hansson2017-07-092-0/+10
|
* Instrument and log the servicesDavid Heinemeier Hansson2017-07-097-32/+169
|
* Merge pull request #27 from dixpac/fix_configuration_keys_symbolizationDavid Heinemeier Hansson2017-07-091-1/+1
|\ | | | | Symbolize all keys inside configuration nested hash
| * Symbolize all keys inside configuration nested hashdixpac2017-07-091-1/+1
| | | | | | | | | | Since configuration is a nested hash we need to symbolize all keys of the hash. Othervise fetcing will fail on start
* | Example of keyfile specificationDavid Heinemeier Hansson2017-07-091-1/+12
| |
* | Ensure binary encoding for downloadingDavid Heinemeier Hansson2017-07-091-1/+1
|/
* Use class methods scope now that we have multipleDavid Heinemeier Hansson2017-07-091-14/+16
|
* Merge pull request #26 from jeremy/service-configuratorDavid Heinemeier Hansson2017-07-095-38/+39
|\ | | | | Clarify how a service can build other composed services
| * Clarify how a service can build other composed servicesJeremy Daer2017-07-095-38/+39
| | | | | | | | | | | | | | | | | | * Service.build takes the literal YAML config hash for the service and a reference to the Configurator that's doing the building. * Services that compose additional services can use the Configurator to look them up and build them by name. See MirrorService for an example. References #23