Mercurial > hg > orthanc-object-storage
annotate Azure/AzureBlobStoragePlugin.cpp @ 89:62752402edbb
azure build for Windows platform
| author | Alain Mazy <alain@mazy.be> |
|---|---|
| date | Mon, 28 Nov 2022 14:57:54 +0100 |
| parents | e29987d80ecf |
| children | 9b2a76fe987a |
| 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 // Create aliases to make the code easier to read. | |
| 30 namespace as = azure::storage; | |
| 31 | |
| 78 | 32 class AzureBlobStoragePlugin : public BaseStorage |
| 1 | 33 { |
| 34 public: | |
| 35 | |
| 36 as::cloud_blob_client blobClient_; | |
| 37 as::cloud_blob_container blobContainer_; | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
38 bool storageContainsUnknownFiles_; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
39 |
| 1 | 40 public: |
| 41 | |
| 79 | 42 AzureBlobStoragePlugin(const std::string& nameForLogs, |
| 43 const as::cloud_blob_client& blobClient, | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
44 const as::cloud_blob_container& blobContainer, |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
45 bool enableLegacyStorageStructure, |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
46 bool storageContainsUnknownFiles |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
47 ); |
| 1 | 48 |
| 49 virtual IWriter* GetWriterForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled); | |
| 50 virtual IReader* GetReaderForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled); | |
| 51 virtual void DeleteObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled); | |
| 52 }; | |
| 53 | |
| 54 | |
| 78 | 55 class Writer : public IStorage::IWriter |
| 1 | 56 { |
| 57 std::string path_; | |
| 58 as::cloud_blob_client client_; | |
| 59 as::cloud_blob_container container_; | |
| 60 | |
| 61 public: | |
| 62 Writer(const as::cloud_blob_container& container, const std::string& path, const as::cloud_blob_client& client) | |
| 63 : path_(path), | |
| 64 client_(client), | |
| 65 container_(container) | |
| 66 { | |
| 67 } | |
| 68 | |
| 69 virtual ~Writer() | |
| 70 { | |
| 71 } | |
| 72 | |
| 73 virtual void Write(const char* data, size_t size) | |
| 74 { | |
| 75 try | |
| 76 { | |
| 77 concurrency::streams::istream inputStream = concurrency::streams::rawptr_stream<uint8_t>::open_istream(reinterpret_cast<const uint8_t*>(data), size); | |
| 34 | 78 azure::storage::cloud_block_blob blob = container_.get_block_blob_reference(utility::conversions::to_string_t(path_)); |
| 1 | 79 blob.upload_from_stream(inputStream); |
| 80 inputStream.close().wait(); | |
| 81 } | |
| 82 catch (std::exception& ex) | |
| 83 { | |
| 84 throw StoragePluginException("AzureBlobStorage: error writing file " + std::string(path_) + ": " + ex.what()); | |
| 85 } | |
| 86 } | |
| 87 }; | |
| 88 | |
| 89 | |
| 78 | 90 class Reader : public IStorage::IReader |
| 1 | 91 { |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
92 std::string path_; |
| 1 | 93 as::cloud_blob_client client_; |
| 94 as::cloud_blob_container container_; | |
| 95 as::cloud_block_blob block_; | |
| 96 | |
| 97 public: | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
98 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
|
99 : client_(client), |
| 1 | 100 container_(container) |
| 101 { | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
102 std::string firstExceptionMessage; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
103 |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
104 for (auto& path: paths) |
| 1 | 105 { |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
106 try |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
107 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
108 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
|
109 block_.download_attributes(); // to retrieve the properties |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
110 path_ = path; |
| 71 | 111 return; |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
112 } |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
113 catch (std::exception& ex) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
114 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
115 if (firstExceptionMessage.empty()) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
116 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
117 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
|
118 } |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
119 //ignore to retry |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
120 } |
| 1 | 121 } |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
122 throw StoragePluginException(firstExceptionMessage); |
| 1 | 123 } |
| 124 | |
| 125 virtual ~Reader() | |
| 126 { | |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
127 } |
| 1 | 128 |
| 129 virtual size_t GetSize() | |
| 130 { | |
| 131 try | |
| 132 { | |
| 133 return block_.properties().size(); | |
| 134 } | |
| 135 catch (std::exception& ex) | |
| 136 { | |
| 137 throw StoragePluginException("AzureBlobStorage: error while reading file " + std::string(path_) + ": " + ex.what()); | |
| 138 } | |
| 139 } | |
| 140 | |
|
39
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
141 virtual void ReadWhole(char* data, size_t size) |
| 1 | 142 { |
| 143 try | |
| 144 { | |
| 145 concurrency::streams::ostream outputStream = concurrency::streams::rawptr_stream<uint8_t>::open_ostream(reinterpret_cast<uint8_t*>(data), size); | |
| 146 | |
| 147 block_.download_to_stream(outputStream); | |
| 148 } | |
| 149 catch (std::exception& ex) | |
| 150 { | |
| 151 throw StoragePluginException("AzureBlobStorage: error while reading file " + std::string(path_) + ": " + ex.what()); | |
| 152 } | |
| 153 } | |
| 154 | |
|
39
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
155 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
|
156 { |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
157 try |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
158 { |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
159 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
|
160 |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
161 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
|
162 } |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
163 catch (std::exception& ex) |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
164 { |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
165 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
|
166 } |
|
50d0be413c42
implemented ReadRange (only in Azure plugin right now)
Alain Mazy <am@osimis.io>
parents:
37
diff
changeset
|
167 } |
| 1 | 168 }; |
| 169 | |
| 170 | |
| 171 | |
| 172 const char* AzureBlobStoragePluginFactory::GetStoragePluginName() | |
| 173 { | |
| 174 return "Azure Blob Storage"; | |
| 175 } | |
| 176 | |
| 48 | 177 bool IsSasTokenAccountLevel(utility::string_t sasToken) |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
178 { |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
179 // Use cpprestsdk's utility::string_t here since the expected argument is the return value of |
| 89 | 180 // as::storage_credentials::sas_token(), which is type utility::string_t (which is a std::wstring on Windows and a std::string on Linux) |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
181 size_t newIdx = 0; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
182 size_t prevIdx = 0; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
183 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
|
184 { |
| 48 | 185 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
|
186 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
|
187 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
188 size_t equalsIdx = kvpair.find('='); |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
189 utility::string_t key = kvpair.substr(0, equalsIdx); |
| 89 | 190 #ifdef WIN32 |
| 191 const wchar_t* srt = L"srt"; | |
| 192 #else | |
| 193 const char* srt = "srt"; | |
| 194 #endif | |
| 195 // if (key == utility::string_t(srt, srt + strlen(srt))) // only account SAS has this parameter | |
| 196 if (key == srt) // only account SAS has this parameter | |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
197 return true; |
| 89 | 198 |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
199 } |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
200 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
201 return false; |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
202 } |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
203 |
| 79 | 204 IStorage* AzureBlobStoragePluginFactory::CreateStorage(const std::string& nameForLogs, const OrthancPlugins::OrthancConfiguration& orthancConfig) |
| 1 | 205 { |
| 206 std::string connectionString; | |
| 207 std::string containerName; | |
| 15 | 208 bool enableLegacyStorageStructure; |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
209 bool storageContainsUnknownFiles; |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
210 bool createContainerIfNotExists; |
| 1 | 211 |
| 81 | 212 if (orthancConfig.IsSection(GetConfigurationSectionName())) |
| 1 | 213 { |
| 214 OrthancPlugins::OrthancConfiguration pluginSection; | |
| 81 | 215 orthancConfig.GetSection(pluginSection, GetConfigurationSectionName()); |
| 1 | 216 |
| 78 | 217 if (!BaseStorage::ReadCommonConfiguration(enableLegacyStorageStructure, storageContainsUnknownFiles, pluginSection)) |
| 15 | 218 { |
| 219 return nullptr; | |
| 220 } | |
| 221 | |
| 1 | 222 if (!pluginSection.LookupStringValue(connectionString, "ConnectionString")) |
| 223 { | |
| 224 OrthancPlugins::LogError("AzureBlobStorage/ConnectionString configuration missing. Unable to initialize plugin"); | |
| 225 return nullptr; | |
| 226 } | |
| 227 | |
| 228 if (!pluginSection.LookupStringValue(containerName, "ContainerName")) | |
| 229 { | |
| 230 OrthancPlugins::LogError("AzureBlobStorage/ContainerName configuration missing. Unable to initialize plugin"); | |
| 231 return nullptr; | |
| 232 } | |
| 233 | |
|
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
|
234 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
|
235 boost::trim(containerName); |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
236 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
237 createContainerIfNotExists = pluginSection.GetBooleanValue("CreateContainerIfNotExists", true); |
| 1 | 238 } |
| 239 else if (orthancConfig.IsSection("BlobStorage")) // backward compatibility with the old plugin: | |
| 240 { | |
| 241 OrthancPlugins::LogWarning("AzureBlobStorage: you're using an old configuration format for the plugin."); | |
| 242 | |
| 243 OrthancPlugins::OrthancConfiguration pluginSection; | |
| 244 orthancConfig.GetSection(pluginSection, "BlobStorage"); | |
| 245 | |
| 246 std::string accountName; | |
| 247 std::string accountKey; | |
| 248 | |
| 249 if (!pluginSection.LookupStringValue(containerName, "ContainerName")) | |
| 250 { | |
| 251 OrthancPlugins::LogError("BlobStorage/AccountName configuration missing. Unable to initialize plugin"); | |
| 252 return nullptr; | |
| 253 } | |
| 254 | |
| 255 if (!pluginSection.LookupStringValue(accountName, "AccountName")) | |
| 256 { | |
| 257 OrthancPlugins::LogError("BlobStorage/AccountName configuration missing. Unable to initialize plugin"); | |
| 258 return nullptr; | |
| 259 } | |
| 260 | |
| 261 if (!pluginSection.LookupStringValue(accountKey, "AccountKey")) | |
| 262 { | |
| 263 OrthancPlugins::LogError("BlobStorage/ContainerName configuration missing. Unable to initialize plugin"); | |
| 264 return nullptr; | |
| 265 } | |
| 266 | |
| 267 std::ostringstream connectionStringBuilder; | |
| 268 connectionStringBuilder << "DefaultEndpointsProtocol=https;AccountName=" << accountName << ";AccountKey=" << accountKey; | |
| 269 connectionString = connectionStringBuilder.str(); | |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
270 |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
271 createContainerIfNotExists = pluginSection.GetBooleanValue("CreateContainerIfNotExists", true); |
| 1 | 272 } |
| 273 else | |
| 274 { | |
| 275 OrthancPlugins::LogWarning(std::string(GetStoragePluginName()) + " plugin, section missing. Plugin is not enabled."); | |
| 276 return nullptr; | |
| 277 } | |
| 278 | |
| 279 try | |
| 280 { | |
| 11 | 281 OrthancPlugins::LogInfo("Connecting to Azure storage ..."); |
| 282 | |
| 34 | 283 as::cloud_storage_account storageAccount = as::cloud_storage_account::parse(utility::conversions::to_string_t(connectionString)); |
| 11 | 284 OrthancPlugins::LogInfo("Storage account created"); |
| 1 | 285 |
| 286 as::cloud_blob_client blobClient = storageAccount.create_cloud_blob_client(); | |
| 11 | 287 OrthancPlugins::LogInfo("Blob client created"); |
| 288 | |
| 34 | 289 as::cloud_blob_container blobContainer = blobClient.get_container_reference(utility::conversions::to_string_t(containerName)); |
| 11 | 290 OrthancPlugins::LogInfo("Accessing blob container"); |
| 1 | 291 |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
292 // 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
|
293 // was used in the connectionString. |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
294 // 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
|
295 // through accountName/accountKey combo or account SAS. |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
296 if ((storageAccount.credentials().is_account_key() || |
| 48 | 297 (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
|
298 && createContainerIfNotExists) |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
299 { |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
300 // 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
|
301 bool containerCreated = blobContainer.create_if_not_exists(); |
| 1 | 302 |
|
46
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
303 if (containerCreated) |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
304 { |
|
3b8fab63313d
Add "CreateContainerIfNotExists" config for Azure
Mark Poscablo <Mark.Poscablo@varian.com>
parents:
15
diff
changeset
|
305 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
|
306 } |
| 1 | 307 } |
| 308 | |
| 309 OrthancPlugins::LogInfo("Blob storage initialized"); | |
| 310 | |
| 79 | 311 return new AzureBlobStoragePlugin(nameForLogs, blobClient, blobContainer, enableLegacyStorageStructure, storageContainsUnknownFiles); |
| 1 | 312 } |
| 313 catch (const std::exception& e) | |
| 314 { | |
| 315 OrthancPlugins::LogError(std::string("AzureBlobStorage plugin: failed to initialize plugin: ") + e.what()); | |
| 316 return nullptr; | |
| 317 } | |
| 318 | |
| 319 } | |
| 320 | |
| 79 | 321 AzureBlobStoragePlugin::AzureBlobStoragePlugin(const std::string& nameForLogs, const as::cloud_blob_client& blobClient, const as::cloud_blob_container& blobContainer, bool enableLegacyStorageStructure, bool storageContainsUnknownFiles) |
| 322 : BaseStorage(nameForLogs, enableLegacyStorageStructure), | |
| 15 | 323 blobClient_(blobClient), |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
324 blobContainer_(blobContainer), |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
325 storageContainsUnknownFiles_(storageContainsUnknownFiles) |
| 1 | 326 { |
| 327 | |
| 328 } | |
| 329 | |
| 78 | 330 IStorage::IWriter* AzureBlobStoragePlugin::GetWriterForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled) |
| 1 | 331 { |
| 332 return new Writer(blobContainer_, GetPath(uuid, type, encryptionEnabled), blobClient_); | |
| 333 } | |
| 334 | |
| 78 | 335 IStorage::IReader* AzureBlobStoragePlugin::GetReaderForObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled) |
| 1 | 336 { |
|
70
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
337 std::list<std::string> paths; |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
338 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
|
339 if (storageContainsUnknownFiles_) |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
340 { |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
341 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
|
342 } |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
343 |
|
16e419fe80c5
Google & Azure: Added the "EnableLegacyUnknownFiles" configuration option
Alain Mazy <am@osimis.io>
parents:
56
diff
changeset
|
344 return new Reader(blobContainer_, paths, blobClient_); |
| 1 | 345 } |
| 346 | |
| 347 void AzureBlobStoragePlugin::DeleteObject(const char* uuid, OrthancPluginContentType type, bool encryptionEnabled) | |
| 348 { | |
| 349 std::string path = GetPath(uuid, type, encryptionEnabled); | |
| 350 | |
| 351 try | |
| 352 { | |
| 34 | 353 as::cloud_block_blob blockBlob = blobContainer_.get_block_blob_reference(utility::conversions::to_string_t(path)); |
| 1 | 354 |
| 355 blockBlob.delete_blob(); | |
| 356 } | |
| 357 catch (std::exception& ex) | |
| 358 { | |
| 359 throw StoragePluginException("AzureBlobStorage: error while deleting file " + std::string(path) + ": " + ex.what()); | |
| 360 } | |
| 361 } |