Bug 56909

Summary: Раздражающая несправедливость при сохранении pve-spice.vv файла в браузере Firefox
Product: Sisyphus Reporter: v.slugin
Component: pve-managerAssignee: Alexey Shabalin <shaba>
Status: NEW --- QA Contact: qa-sisyphus
Severity: normal    
Priority: P5 CC: darisishe, shaba, shrek, tyaplyapych, vsu
Version: unstable   
Hardware: x86_64   
OS: Linux   
Attachments:
Description Flags
результат после отключения проверки браузеров none

Description v.slugin 2025-11-18 21:21:25 MSK 
Created attachment 20112 [details]
результат после отключения проверки браузеров

На браузерах, попадающих под этот фильтр, файл сохраняется как надо:

            // Note: we need to tell Android, AppleWebKit and Chrome
            // the correct file name extension
            // but we do not set 'download' tag for other environments, because
            // It can have strange side effects (additional user prompt on firefox)
    
          if (navigator.userAgent.match(/Android|AppleWebKit|Chrome/i)) {
                link.download = name;
          }


Однако браузера Firefox в регулярном выражении проверки нет. Следовательно, в Firefox файл сохраняется без расширения .vv и имя задаётся случайным образом. Что не может не раздражать, когда пытаешься связать с таким типом загружаемого через браузер файла приложение.

Вопреки [It can have strange side effects (additional user prompt on firefox)] мой Firefox загружает всё корректно без всяких [additional user prompt], поэтому предлагаю на уровне альтовой версии pve выпилить эту проверку. Ну или дополнить регулярное выражение.

Я локально чинил так:

1. nano +2926 /usr/share/pve-manager/js/pvemanagerlib.js
2. закомментировать строки 2926 и 2928
3. сохранить файл
4. сбросить кеш в браузере
Comment 1 Sergey Vlasov 2025-11-18 21:49:21 MSK 
Судя по https://github.com/proxmox/pve-manager/blame/8058410f40568d46d323cd6ad3b17c3327789594/www/manager6/Utils.js#L1254, изначально этому куску кода более 10 лет; с тех пор туда добавили Chrome (https://github.com/proxmox/pve-manager/commit/63c2296675a362f1f8f83ff1cf55ace2b752b3d6) и AppleWebKit (https://github.com/proxmox/pve-manager/commit/e8ed907b7e3266450750ea6f4125857f3129325c).  Есть существенная вероятность, что в Firefox мешавшее здесь поведение уже давным-давно исправлено.