diff options
| author | Cameron Bothner <cameronbothner@gmail.com> | 2018-08-23 23:36:43 -0400 |
|---|---|---|
| committer | Cameron Bothner <cameronbothner@gmail.com> | 2018-08-23 23:36:43 -0400 |
| commit | aae56c35290da2a6251b773b3f93845e21791823 (patch) | |
| tree | 7cc21e735b4bc2845228f084528fc1b66531ab5d /activestorage/CHANGELOG.md | |
| parent | 3868648cae36fd64741135e3d33d7055e925879b (diff) | |
| download | rails-aae56c35290da2a6251b773b3f93845e21791823.tar.gz rails-aae56c35290da2a6251b773b3f93845e21791823.tar.bz2 rails-aae56c35290da2a6251b773b3f93845e21791823.zip | |
Handle only specifically relevant Azure HTTPErrors
The Azure gem uses `Azure::Core::Http::HTTPError` for everything:
checksum mismatch, missing object, network unavailable, and many more.
(https://www.rubydoc.info/github/yaxia/azure-storage-ruby/Azure/Core/Http/HTTPError).
Rescuing that class obscures all sorts of configuration errors. We
should check the type of error in those rescue blocks, and reraise when
needed.
Diffstat (limited to 'activestorage/CHANGELOG.md')
| -rw-r--r-- | activestorage/CHANGELOG.md | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/activestorage/CHANGELOG.md b/activestorage/CHANGELOG.md index b592f79ca6..92e300a440 100644 --- a/activestorage/CHANGELOG.md +++ b/activestorage/CHANGELOG.md @@ -1,3 +1,10 @@ +* `ActiveStorage::Service::AzureStorageService` only handles specifically + relevant types of `Azure::Core::Http::HTTPError`. It previously obscured + other types of `HTTPError`, which is the azure-storage gem’s catch-all + exception class. + + *Cameron Bothner* + * `ActiveStorage::DiskController#show` generates a 404 Not Found response when the requested file is missing from the disk service. It previously raised `Errno::ENOENT`. |