[ 
https://issues.apache.org/jira/browse/LIBCLOUD-458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16122826#comment-16122826
 ] 

ASF GitHub Bot commented on LIBCLOUD-458:
-----------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/libcloud/pull/1076


> Use HTTPS CDN URIs for Rackspace Cloud Files
> --------------------------------------------
>
>                 Key: LIBCLOUD-458
>                 URL: https://issues.apache.org/jira/browse/LIBCLOUD-458
>             Project: Libcloud
>          Issue Type: Improvement
>          Components: Storage
>    Affects Versions: 0.14.0-beta3
>            Reporter: Michael Farrell
>
> I found another patch in my collection, where I had patched the Rackspace 
> storage driver to use the HTTPS CDN URI.
> I'm unsure if it may be best to have this as an option for storage drivers, 
> but if this option is added, it will also need to be implemented for the 
> other storage drivers.  I would like some advice on this.
> The patch that hard codes it to use HTTPS URIs always for the CDN, as a 
> 1-line change (ie: without the settings proposed above):
> {code}
> -------------------- libcloud/storage/drivers/cloudfiles.py 
> --------------------
> index 1502a33..6e7f931 100644
> @@ -273,7 +273,7 @@ class CloudFilesStorageDriver(StorageDriver, 
> OpenStackDriverMixin):
>                                             cdn_request=True)
>  
>          if response.status == httplib.NO_CONTENT:
> -            cdn_url = response.headers['x-cdn-uri']
> +            cdn_url = response.headers['x-cdn-ssl-uri']
>              return cdn_url
>          elif response.status == httplib.NOT_FOUND:
>              raise ContainerDoesNotExistError(value='',
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to