Kami commented on a change in pull request #1408: Implement SAS URL generation
for Azure Storage
URL: https://github.com/apache/libcloud/pull/1408#discussion_r366147905
##########
File path: libcloud/storage/drivers/azure_blobs.py
##########
@@ -489,6 +503,70 @@ def get_object(self, container_name, object_name):
raise ObjectDoesNotExistError(value=None, driver=self,
object_name=object_name)
+ def get_object_cdn_url(self, obj,
+ ex_expiry=AZURE_STORAGE_CDN_URL_EXPIRY_HOURS):
Review comment:
Going forward, we can also start adding type annotations for various method
arguments in the drivers when we introduce new functionality.
We now don't support Python 2.7 anymore, but perhaps we should still start
with type annotations in the comments and move to putting them directly into
the method signatures in the future.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
