Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Use class methods scope now that we have multiple | David Heinemeier Hansson | 2017-07-09 | 1 | -14/+16 |
| | |||||
* | Merge pull request #26 from jeremy/service-configurator | David Heinemeier Hansson | 2017-07-09 | 5 | -38/+39 |
|\ | | | | | Clarify how a service can build other composed services | ||||
| * | Clarify how a service can build other composed services | Jeremy Daer | 2017-07-09 | 5 | -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 | ||||
* | | Match new configurator needs | David Heinemeier Hansson | 2017-07-09 | 1 | -0/+2 |
|/ | |||||
* | Service: clarify Service.build arguments | Jeremy Daer | 2017-07-09 | 2 | -5/+5 |
| | | | | | | | First arg is config for the service we're instantiating. Second arg is service configurations so we can look up and configure other services by name. | ||||
* | Travis CI 💡 | Jeremy Daer | 2017-07-08 | 1 | -0/+15 |
| | |||||
* | Configure services that reference other services | Jeremy Daer | 2017-07-08 | 10 | -43/+89 |
| | | | | | | | | * Move service configuration from the Engine to Service * Delegate configuration mechanics to internal Service::Configurator * Delegate service building to the concrete Service classes, allowing them to configure composed services. * Implement for the Mirror service. | ||||
* | Mirror: explicit primary service and list of mirrors | Jeremy Daer | 2017-07-08 | 2 | -15/+22 |
| | | | | | | | | | | | | Pass separate primary service and list of mirrors rather than treating the first of the services list as the primary. Nice fit for keyword args, and something we've long wanted in the equivalent Basecamp file repository. Upload returns the results of the underlying service uploads rather than the io.rewind result. Rewind before uploading rather than afterward, and demonstrate that behavior with a test. Test that more than one mirror works. | ||||
* | Merge pull request #21 from jeremy/disk-binary-io | David Heinemeier Hansson | 2017-07-09 | 2 | -14/+8 |
|\ | | | | | Disk service: use binary IO throughout, not UTF-8 | ||||
| * | Disk service: use binary IO throughout, not UTF-8 | Jeremy Daer | 2017-07-08 | 2 | -14/+8 |
| | | |||||
* | | Merge pull request #22 from jeremy/disk-filename-url | David Heinemeier Hansson | 2017-07-09 | 5 | -9/+9 |
|\ \ | |/ |/| | Disk storage: ensure URLs end with the blob filename | ||||
| * | Disk storage: ensure URLs end with the blob filename since some user agents ↵ | Jeremy Daer | 2017-07-08 | 5 | -9/+9 |
|/ | | | | don't respect Content-Disposition filename | ||||
* | Tests: skip GCS/AWS service tests if unconfigured | Jeremy Daer | 2017-07-08 | 1 | -0/+1 |
| | |||||
* | Merge pull request #15 from robin850/documentation | David Heinemeier Hansson | 2017-07-08 | 6 | -5/+100 |
|\ | | | | | Add some documentation | ||||
| * | Add some documentation | Robin Dupret | 2017-07-07 | 6 | -5/+100 |
| | | |||||
* | | Add a brief roadmap section | David Heinemeier Hansson | 2017-07-08 | 1 | -1/+5 |
| | | |||||
* | | Change type of created_at columns from time to datetime | George Claghorn | 2017-07-07 | 1 | -8/+8 |
| | | | | | | | | We intend to store a date and time, not merely a time. | ||||
* | | It is an engine (because of tasks) not a railtie | David Heinemeier Hansson | 2017-07-07 | 2 | -2/+2 |
| | | |||||
* | | Merge pull request #14 from maratgaliev/readme_patch | David Heinemeier Hansson | 2017-07-07 | 1 | -1/+1 |
|\ \ | |/ |/| | Update README.md | ||||
| * | Update README.md | Marat Galiev | 2017-07-07 | 1 | -1/+1 |
|/ | | | For the first look It's obvious, but makes sense I think, and raises undefined method `active_storage' on migration run for example. | ||||
* | Bundle google-cloud-storage instead of the full Google SDK | George Claghorn | 2017-07-07 | 2 | -91/+2 |
| | |||||
* | Create db/migrate if it doesn't exist | George Claghorn | 2017-07-06 | 1 | -0/+1 |
| | |||||
* | Merge pull request #10 from bradly/fix-readme-typo | George Claghorn | 2017-07-06 | 1 | -1/+1 |
|\ | | | | | Fixing typo in Readme | ||||
| * | Fixing typo in Readme | Bradly Feeley | 2017-07-06 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #8 from robin850/fix-migrations | George Claghorn | 2017-07-06 | 3 | -4/+4 |
|\ \ | |/ |/| | Fix the migration class name | ||||
| * | Fix the migration class name | Robin Dupret | 2017-07-06 | 3 | -4/+4 |
| | | | | | | | | | | | | Due to Active Support auto loading feature, the migration class shouldn't be name-spaced under the `ActiveStorage` constant, otherwise, running the migrations would throw an error. | ||||
* | | Test deleting a nonexistent key | George Claghorn | 2017-07-06 | 1 | -0/+6 |
| | | |||||
* | | Use safe navigation | George Claghorn | 2017-07-06 | 1 | -1/+1 |
| | | |||||
* | | Remove unnecessary method delegations | George Claghorn | 2017-07-06 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #7 from johnwilliams/test_config_error_typo | George Claghorn | 2017-07-06 | 1 | -1/+1 |
|\ \ | |/ |/| | Correct config path in error message | ||||
| * | Correct config path in error message | John Williams | 2017-07-06 | 1 | -1/+1 |
|/ | |||||
* | Fix test | George Claghorn | 2017-07-06 | 2 | -2/+4 |
| | | | | | | | * S3 fails fast if the Content-MD5 header on an upload request is an invalid checksum. Send a valid but incorrect checksum. * Rescue the service-specific exception and raise the generic one. | ||||
* | Merge pull request #6 from johnwilliams/s3_integrity | David Heinemeier Hansson | 2017-07-06 | 1 | -2/+1 |
|\ | | | | | Ensure S3 file integrity by sending checksum | ||||
| * | Merge branch 'master' into s3_integrity | John Williams | 2017-07-06 | 1 | -2/+3 |
| |\ | |/ |/| | |||||
* | | Style | George Claghorn | 2017-07-06 | 1 | -5/+3 |
| | | |||||
* | | Merge pull request #5 from SGospodinov/master | George Claghorn | 2017-07-06 | 1 | -2/+5 |
|\ \ | | | | | | | Adding server-side integrity check for GCS Service | ||||
| * | | Adding server side integrity check for GCS Service | Stanislav Gospodinov | 2017-07-06 | 1 | -2/+5 |
|/ / | |||||
| * | Send checksum to S3 to verify file integrity | John Williams | 2017-07-06 | 1 | -2/+1 |
|/ | |||||
* | Merge pull request #4 from snuggs/patch-1 | George Claghorn | 2017-07-06 | 1 | -1/+1 |
|\ | | | | | Fix hash usage consistency. | ||||
| * | Fix hash usage consistency. | Ra'Shaun Stovall | 2017-07-06 | 1 | -1/+1 |
|/ | | | | | | | Unless this was intentional, being consistent with: https://github.com/rails/activestorage/blob/master/lib/active_storage/service/s3_service.rb#L8 Just showin' a lil' <3 while perusing the repo @dhh | ||||
* | Link up main models | David Heinemeier Hansson | 2017-07-06 | 1 | -1/+1 |
| | |||||
* | Describe some of the design differences in AS | David Heinemeier Hansson | 2017-07-06 | 1 | -0/+6 |
| | |||||
* | Merge pull request #3 from adrianpacala/format-readme-examples | David Heinemeier Hansson | 2017-07-06 | 1 | -1/+5 |
|\ | | | | | Use correct syntax highlighting in erb block | ||||
| * | Use correct syntax in erb block | Adrian Pacała | 2017-07-06 | 1 | -1/+5 |
|/ | |||||
* | Documentation, yo! | David Heinemeier Hansson | 2017-07-06 | 1 | -0/+1 |
| | |||||
* | Helper methods are private | David Heinemeier Hansson | 2017-07-06 | 1 | -4/+5 |
| | |||||
* | Check integrity after uploads | David Heinemeier Hansson | 2017-07-06 | 7 | -9/+34 |
| | |||||
* | Escape commands and paths | David Heinemeier Hansson | 2017-07-06 | 1 | -3/+3 |
| | |||||
* | Dropped from the interface | David Heinemeier Hansson | 2017-07-06 | 1 | -9/+0 |
| | |||||
* | It's base64 now since the clouds expect that | David Heinemeier Hansson | 2017-07-06 | 1 | -1/+1 |
| | | | | Gotta please them clouds. SPEAK THE CLOUD. |