[jira] [Updated] (CLOUDSTACK-7312) ISO/volume format name checking is crude and doesn't work with advanced URLs

["Min Chen (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/CLOUDSTACK-7312?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Min Chen updated CLOUDSTACK-7312:
---------------------------------

    Description: 
When registering ISO/Volume with CloudStack, it is necessary to provide a URL from which the ISO/Volume can be downloaded.

SO/Volume name checking currently just looks at the very end of the url string. e.g.:

private void checkFormat(String format, String url) {
if((!url.toLowerCase().endsWith("vhd"))

This breaks functionality for  S3 pre-signed URL, or anything where the file extension is not the last part of the URL. We should at least attempt to parse the URL for filename vs parameters.


  was:
When registering ISOs with CloudStack, it is necessary to provide a URL from which the ISO can be downloaded.

CloudStack expects and requires the URL to carry a file extension that matches the expected iso, eg. ".iso", "iso.zip", "iso.bz2", "iso.gz". If the URL doesn't have such an extension, it will be rejected, even if it is a perfectly valid URL from which an ISO can be downloaded.


        Summary: ISO/volume format name checking is crude and doesn't work with advanced URLs  (was: ISOs cannot be downloaded from URLs without matching file extensions)

> ISO/volume format name checking is crude and doesn't work with advanced URLs
> ----------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-7312
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-7312
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the default.) 
>          Components: Storage Controller
>    Affects Versions: 4.2.0
>            Reporter: Min Chen
>            Assignee: Min Chen
>             Fix For: 4.5.0
>
>
> When registering ISO/Volume with CloudStack, it is necessary to provide a URL from which the ISO/Volume can be downloaded.
> SO/Volume name checking currently just looks at the very end of the url string. e.g.:
> private void checkFormat(String format, String url) {
> if((!url.toLowerCase().endsWith("vhd"))
> This breaks functionality for  S3 pre-signed URL, or anything where the file extension is not the last part of the URL. We should at least attempt to parse the URL for filename vs parameters.



--
This message was sent by Atlassian JIRA
(v6.2#6252)