[pve-devel] [PATCH manager] ui: allow decompressing ISO compressed with bz2

[Aaron Lauterer]

Tested in combination with the adjacent backend patch [0] by Downloading 
a bz2 compressed ISO and booting a VM from it.

One small style nit inline, but I don't think that warants a new 
version, as we can fix it in a follow up or just directly when applying 
the patch.

[0] 
https://lore.proxmox.com/pve-devel/d92e4078-00bf-48ad-9be8-afaeaf7398be@proxmox.com/T/#m6cff68830fd5356ebddfff8cee7f4b8c7d7770bd

Tested-By: Aaron Lauterer <a.lauterer at proxmox.com>
Reviewed-By: Aaron Lauterer <a.lauterer at proxmox.com>

On  2024-08-02  16:37, Maximiliano Sandoval wrote:
> Signed-off-by: Maximiliano Sandoval <m.sandoval at proxmox.com>
> ---
> 
> This is a follow-up of the patch:
> 
>      [PATCH storage] storage: add bzip2 support
> 
> which can be found at [1].
> 
> [1] https://lists.proxmox.com/pipermail/pve-devel/2024-August/065030.html
> 
> 
>   www/manager6/window/DownloadUrlToStorage.js | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/www/manager6/window/DownloadUrlToStorage.js b/www/manager6/window/DownloadUrlToStorage.js
> index 5523a152..9cd44d5c 100644
> --- a/www/manager6/window/DownloadUrlToStorage.js
> +++ b/www/manager6/window/DownloadUrlToStorage.js
> @@ -84,7 +84,7 @@ Ext.define('PVE.window.DownloadUrlToStorage', {
>   		    let filename = data.filename || "";
>   		    let compression = '__default__';
>   		    if (view.content === 'iso') {
> -			const matches = filename.match(/^(.+)\.(gz|lzo|zst)$/i);
> +			const matches = filename.match(/^(.+)\.(gz|lzo|zst|bz2)$/i);
>   			if (matches) {
>   			    filename = matches[1];
>   			    compression = matches[2].toLowerCase();
> @@ -228,6 +228,7 @@ Ext.define('PVE.window.DownloadUrlToStorage', {
>   				['lzo', 'LZO'],
>   				['gz', 'GZIP'],
>   				['zst', 'ZSTD'],
> +				['bz2', 'bzip2'],

All the other options in the list are all-caps. We probably want this 
also to be BZIP2 to not break the overall theme here.

>   		    ],
>   		    cbind: {
>   			hidden: get => get('content') !== 'iso',