Mercurial > hg > orthanc-object-storage
annotate Azure/AzureBlobStoragePlugin.cpp @ 70:16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
| author | Alain Mazy <am@osimis.io> |
|---|---|
| date | Mon, 06 Sep 2021 14:21:56 +0200 |
| parents | b922ae86bbe1 |
| children | a2bb9acdc5e6 |
| rev | line source |
|---|---|
| 1 | 1 /** |
| 2 * Cloud storage plugins for Orthanc | |
|
37
f55b2afdf53d
upgrade to year 2021
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
34
diff
changeset
|
3 * Copyright (C) 2020-2021 Osimis S.A., Belgium |
| 1 | 4 * |
| 5 * This program is free software: you can redistribute it and/or | |
| 6 * modify it under the terms of the GNU Affero General Public License | |
| 7 * as published by the Free Software Foundation, either version 3 of | |
| 8 * the License, or (at your option) any later version. | |
| 9 * | |
| 10 * This program is distributed in the hope that it will be useful, but | |
| 11 * WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 13 * Affero General Public License for more details. | |
| 14 * | |
| 15 * You should have received a copy of the GNU Affero General Public License | |
| 16 * along with this program. If not, see <http://www.gnu.org/licenses/>. | |
| 17 **/ | |
| 18 | |
|
56
b922ae86bbe1
full static linking against AWS SDK
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
48
diff
changeset
|
19 |
| 1 | 20 #include "AzureBlobStoragePlugin.h" |
| 21 | |
| 22 #include <was/storage_account.h> | |
| 23 #include <was/blob.h> | |
| 24 #include <boost/lexical_cast.hpp> | |
|
12
a203d0a4fd8f
Azure: now removing spaces and CR at the end of the connectionString since it prevented the plugin to initialize.
Alain Mazy
parents:
11
diff
changeset
|
25 #include <boost/algorithm/string.hpp> |
| 1 | 26 #include "cpprest/rawptrstream.h" |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
27 #include "cpprest/details/basic_types.h" |
| 1 | 28 |
| 29 | |
| 30 // Create aliases to make the code easier to read. | |
| 31 namespace as = azure::storage; | |
| 15 | 32 static const char* const PLUGIN_SECTION = "AzureBlobStorage"; |
| 1 | 33 |
| 15 | 34 class AzureBlobStoragePlugin : public BaseStoragePlugin |
| 1 | 35 { |
| 36 public: | |
| 37 | |
| 38 as::cloud_blob_client blobClient_; | |
| 39 as::cloud_blob_container blobContainer_; | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
40 bool storageContainsUnknownFiles_; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
41 |
| 1 | 42 public: |
| 43 | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
44 AzureBlobStoragePlugin(const as::cloud_blob_client& blobClient, |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
45 const as::cloud_blob_container& blobContainer, |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
46 bool enableLegacyStorageStructure, |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
47 bool storageContainsUnknownFiles |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
48 ); |
| 1 | 49 |
| 50 virtual IWriter* GetWriterForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled); | |
| 51 virtual IReader* GetReaderForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled); | |
| 52 virtual void DeleteObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled); | |
| 15 | 53 virtual const char* GetConfigurationSectionName() {return PLUGIN_SECTION;}; |
| 1 | 54 }; |
| 55 | |
| 56 | |
| 57 class Writer : public IStoragePlugin::IWriter | |
| 58 { | |
| 59 std::string path_; | |
| 60 as::cloud_blob_client client_; | |
| 61 as::cloud_blob_container container_; | |
| 62 | |
| 63 public: | |
| 64 Writer(const as::cloud_blob_container& container, const std::string& path, const as::cloud_blob_client& client) | |
| 65 : path_(path), | |
| 66 client_(client), | |
| 67 container_(container) | |
| 68 { | |
| 69 } | |
| 70 | |
| 71 virtual ~Writer() | |
| 72 { | |
| 73 } | |
| 74 | |
| 75 virtual void Write(const char* data, size_t size) | |
| 76 { | |
| 77 try | |
| 78 { | |
| 79 concurrency::streams::istream inputStream = concurrency::streams::rawptr_stream<uint8_t>::open_istream(reinterpret_cast<const uint8_t*>(data), size); | |
| 34 | 80 azure::storage::cloud_block_blob blob = container_.get_block_blob_reference(utility::conversions::to_string_t(path_)); |
| 1 | 81 blob.upload_from_stream(inputStream); |
| 82 inputStream.close().wait(); | |
| 83 } | |
| 84 catch (std::exception& ex) | |
| 85 { | |
| 86 throw StoragePluginException("AzureBlobStorage: error writing file " + std::string(path_) + ": " + ex.what()); | |
| 87 } | |
| 88 } | |
| 89 }; | |
| 90 | |
| 91 | |
| 92 class Reader : public IStoragePlugin::IReader | |
| 93 { | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
94 std::string path_; |
| 1 | 95 as::cloud_blob_client client_; |
| 96 as::cloud_blob_container container_; | |
| 97 as::cloud_block_blob block_; | |
| 98 | |
| 99 public: | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
100 Reader(const as::cloud_blob_container& container, const std::list<std::string>& paths, const as::cloud_blob_client& client) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
101 : client_(client), |
| 1 | 102 container_(container) |
| 103 { | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
104 std::string firstExceptionMessage; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
105 |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
106 for (auto& path: paths) |
| 1 | 107 { |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
108 try |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
109 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
110 block_ = container_.get_block_blob_reference(utility::conversions::to_string_t(path)); |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
111 block_.download_attributes(); // to retrieve the properties |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
112 path_ = path; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
113 } |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
114 catch (std::exception& ex) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
115 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
116 if (firstExceptionMessage.empty()) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
117 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
118 firstExceptionMessage = "AzureBlobStorage: error opening file for reading " + std::string(path) + ": " + ex.what(); |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
119 } |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
120 //ignore to retry |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
121 } |
| 1 | 122 } |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
123 throw StoragePluginException(firstExceptionMessage); |
| 1 | 124 } |
| 125 | |
| 126 virtual ~Reader() | |
| 127 { | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
128 } |
| 1 | 129 |
| 130 virtual size_t GetSize() | |
| 131 { | |
| 132 try | |
| 133 { | |
| 134 return block_.properties().size(); | |
| 135 } | |
| 136 catch (std::exception& ex) | |
| 137 { | |
| 138 throw StoragePluginException("AzureBlobStorage: error while reading file " + std::string(path_) + ": " + ex.what()); | |
| 139 } | |
| 140 } | |
| 141 | |
|
39
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
142 virtual void ReadWhole(char* data, size_t size) |
| 1 | 143 { |
| 144 try | |
| 145 { | |
| 146 concurrency::streams::ostream outputStream = concurrency::streams::rawptr_stream<uint8_t>::open_ostream(reinterpret_cast<uint8_t*>(data), size); | |
| 147 | |
| 148 block_.download_to_stream(outputStream); | |
| 149 } | |
| 150 catch (std::exception& ex) | |
| 151 { | |
| 152 throw StoragePluginException("AzureBlobStorage: error while reading file " + std::string(path_) + ": " + ex.what()); | |
| 153 } | |
| 154 } | |
| 155 | |
|
39
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
156 virtual void ReadRange(char* data, size_t size, size_t fromOffset) |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
157 { |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
158 try |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
159 { |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
160 concurrency::streams::ostream outputStream = concurrency::streams::rawptr_stream<uint8_t>::open_ostream(reinterpret_cast<uint8_t*>(data), size); |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
161 |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
162 block_.download_range_to_stream(outputStream, fromOffset, size); |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
163 } |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
164 catch (std::exception& ex) |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
165 { |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
166 throw StoragePluginException("AzureBlobStorage: error while reading partial file " + std::string(path_) + ": " + ex.what()); |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
167 } |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
168 } |
| 1 | 169 }; |
| 170 | |
| 171 | |
| 172 | |
| 173 const char* AzureBlobStoragePluginFactory::GetStoragePluginName() | |
| 174 { | |
| 175 return "Azure Blob Storage"; | |
| 176 } | |
| 177 | |
| 48 | 178 bool IsSasTokenAccountLevel(utility::string_t sasToken) |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
179 { |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
180 // Use cpprestsdk's utility::string_t here since the expected argument is the return value of |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
181 // as::storage_credentials::sas_token(), which is type utility::string_t |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
182 size_t newIdx = 0; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
183 size_t prevIdx = 0; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
184 while ((newIdx = sasToken.find('&', prevIdx)) != utility::string_t::npos) |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
185 { |
| 48 | 186 utility::string_t kvpair = sasToken.substr(prevIdx, newIdx - prevIdx); |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
187 prevIdx = newIdx + 1; // start next time from char after '&' |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
188 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
189 size_t equalsIdx = kvpair.find('='); |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
190 utility::string_t key = kvpair.substr(0, equalsIdx); |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
191 if (key == "srt") // only account SAS has this parameter |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
192 return true; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
193 } |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
194 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
195 return false; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
196 } |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
197 |
| 1 | 198 IStoragePlugin* AzureBlobStoragePluginFactory::CreateStoragePlugin(const OrthancPlugins::OrthancConfiguration& orthancConfig) |
| 199 { | |
| 200 std::string connectionString; | |
| 201 std::string containerName; | |
| 15 | 202 bool enableLegacyStorageStructure; |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
203 bool storageContainsUnknownFiles; |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
204 bool createContainerIfNotExists; |
| 1 | 205 |
| 206 if (orthancConfig.IsSection(PLUGIN_SECTION)) | |
| 207 { | |
| 208 OrthancPlugins::OrthancConfiguration pluginSection; | |
| 209 orthancConfig.GetSection(pluginSection, PLUGIN_SECTION); | |
| 210 | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
211 if (!BaseStoragePlugin::ReadCommonConfiguration(enableLegacyStorageStructure, storageContainsUnknownFiles, pluginSection)) |
| 15 | 212 { |
| 213 return nullptr; | |
| 214 } | |
| 215 | |
| 1 | 216 if (!pluginSection.LookupStringValue(connectionString, "ConnectionString")) |
| 217 { | |
| 218 OrthancPlugins::LogError("AzureBlobStorage/ConnectionString configuration missing. Unable to initialize plugin"); | |
| 219 return nullptr; | |
| 220 } | |
| 221 | |
| 222 if (!pluginSection.LookupStringValue(containerName, "ContainerName")) | |
| 223 { | |
| 224 OrthancPlugins::LogError("AzureBlobStorage/ContainerName configuration missing. Unable to initialize plugin"); | |
| 225 return nullptr; | |
| 226 } | |
| 227 | |
|
12
a203d0a4fd8f
Azure: now removing spaces and CR at the end of the connectionString since it prevented the plugin to initialize.
Alain Mazy
parents:
11
diff
changeset
|
228 boost::trim(connectionString); // without that, if there's an EOL in the string, it fails with "provided uri is invalid" |
|
a203d0a4fd8f
Azure: now removing spaces and CR at the end of the connectionString since it prevented the plugin to initialize.
Alain Mazy
parents:
11
diff
changeset
|
229 boost::trim(containerName); |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
230 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
231 createContainerIfNotExists = pluginSection.GetBooleanValue("CreateContainerIfNotExists", true); |
| 1 | 232 } |
| 233 else if (orthancConfig.IsSection("BlobStorage")) // backward compatibility with the old plugin: | |
| 234 { | |
| 235 OrthancPlugins::LogWarning("AzureBlobStorage: you're using an old configuration format for the plugin."); | |
| 236 | |
| 237 OrthancPlugins::OrthancConfiguration pluginSection; | |
| 238 orthancConfig.GetSection(pluginSection, "BlobStorage"); | |
| 239 | |
| 240 std::string accountName; | |
| 241 std::string accountKey; | |
| 242 | |
| 243 if (!pluginSection.LookupStringValue(containerName, "ContainerName")) | |
| 244 { | |
| 245 OrthancPlugins::LogError("BlobStorage/AccountName configuration missing. Unable to initialize plugin"); | |
| 246 return nullptr; | |
| 247 } | |
| 248 | |
| 249 if (!pluginSection.LookupStringValue(accountName, "AccountName")) | |
| 250 { | |
| 251 OrthancPlugins::LogError("BlobStorage/AccountName configuration missing. Unable to initialize plugin"); | |
| 252 return nullptr; | |
| 253 } | |
| 254 | |
| 255 if (!pluginSection.LookupStringValue(accountKey, "AccountKey")) | |
| 256 { | |
| 257 OrthancPlugins::LogError("BlobStorage/ContainerName configuration missing. Unable to initialize plugin"); | |
| 258 return nullptr; | |
| 259 } | |
| 260 | |
| 261 std::ostringstream connectionStringBuilder; | |
| 262 connectionStringBuilder << "DefaultEndpointsProtocol=https;AccountName=" << accountName << ";AccountKey=" << accountKey; | |
| 263 connectionString = connectionStringBuilder.str(); | |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
264 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
265 createContainerIfNotExists = pluginSection.GetBooleanValue("CreateContainerIfNotExists", true); |
| 1 | 266 } |
| 267 else | |
| 268 { | |
| 269 OrthancPlugins::LogWarning(std::string(GetStoragePluginName()) + " plugin, section missing. Plugin is not enabled."); | |
| 270 return nullptr; | |
| 271 } | |
| 272 | |
| 273 try | |
| 274 { | |
| 11 | 275 OrthancPlugins::LogInfo("Connecting to Azure storage ..."); |
| 276 | |
| 34 | 277 as::cloud_storage_account storageAccount = as::cloud_storage_account::parse(utility::conversions::to_string_t(connectionString)); |
| 11 | 278 OrthancPlugins::LogInfo("Storage account created"); |
| 1 | 279 |
| 280 as::cloud_blob_client blobClient = storageAccount.create_cloud_blob_client(); | |
| 11 | 281 OrthancPlugins::LogInfo("Blob client created"); |
| 282 | |
| 34 | 283 as::cloud_blob_container blobContainer = blobClient.get_container_reference(utility::conversions::to_string_t(containerName)); |
| 11 | 284 OrthancPlugins::LogInfo("Accessing blob container"); |
| 1 | 285 |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
286 // blobContainer.create_if_not_exists() throws an error if a service SAS (for a blob container) |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
287 // was used in the connectionString. |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
288 // Only allow the use of this function when using storage account-level credentials, whether |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
289 // through accountName/accountKey combo or account SAS. |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
290 if ((storageAccount.credentials().is_account_key() || |
| 48 | 291 (storageAccount.credentials().is_sas() && IsSasTokenAccountLevel(storageAccount.credentials().sas_token()))) |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
292 && createContainerIfNotExists) |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
293 { |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
294 // Return value is true if the container did not exist and was successfully created. |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
295 bool containerCreated = blobContainer.create_if_not_exists(); |
| 1 | 296 |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
297 if (containerCreated) |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
298 { |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
299 OrthancPlugins::LogWarning("Blob Storage Area container has been created. **** check in the Azure console that your container is private ****"); |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
300 } |
| 1 | 301 } |
| 302 | |
| 303 OrthancPlugins::LogInfo("Blob storage initialized"); | |
| 304 | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
305 return new AzureBlobStoragePlugin(blobClient, blobContainer, enableLegacyStorageStructure, storageContainsUnknownFiles); |
| 1 | 306 } |
| 307 catch (const std::exception& e) | |
| 308 { | |
| 309 OrthancPlugins::LogError(std::string("AzureBlobStorage plugin: failed to initialize plugin: ") + e.what()); | |
| 310 return nullptr; | |
| 311 } | |
| 312 | |
| 313 } | |
| 314 | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
315 AzureBlobStoragePlugin::AzureBlobStoragePlugin(const as::cloud_blob_client& blobClient, const as::cloud_blob_container& blobContainer, bool enableLegacyStorageStructure, bool storageContainsUnknownFiles) |
| 15 | 316 : BaseStoragePlugin(enableLegacyStorageStructure), |
| 317 blobClient_(blobClient), | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
318 blobContainer_(blobContainer), |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
319 storageContainsUnknownFiles_(storageContainsUnknownFiles) |
| 1 | 320 { |
| 321 | |
| 322 } | |
| 323 | |
| 324 IStoragePlugin::IWriter* AzureBlobStoragePlugin::GetWriterForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled) | |
| 325 { | |
| 326 return new Writer(blobContainer_, GetPath(uuid, type, encryptionEnabled), blobClient_); | |
| 327 } | |
| 328 | |
| 329 IStoragePlugin::IReader* AzureBlobStoragePlugin::GetReaderForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled) | |
| 330 { | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
331 std::list<std::string> paths; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
332 paths.push_back(GetPath(uuid, type, encryptionEnabled, false)); |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
333 if (storageContainsUnknownFiles_) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
334 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
335 paths.push_back(GetPath(uuid, type, encryptionEnabled, true)); |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
336 } |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
337 |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
338 return new Reader(blobContainer_, paths, blobClient_); |
| 1 | 339 } |
| 340 | |
| 341 void AzureBlobStoragePlugin::DeleteObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled) | |
| 342 { | |
| 343 std::string path = GetPath(uuid, type, encryptionEnabled); | |
| 344 | |
| 345 try | |
| 346 { | |
| 34 | 347 as::cloud_block_blob blockBlob = blobContainer_.get_block_blob_reference(utility::conversions::to_string_t(path)); |
| 1 | 348 |
| 349 blockBlob.delete_blob(); | |
| 350 } | |
| 351 catch (std::exception& ex) | |
| 352 { | |
| 353 throw StoragePluginException("AzureBlobStorage: error while deleting file " + std::string(path) + ": " + ex.what()); | |
| 354 } | |
| 355 } |