Bug 56909 - Раздражающая несправедливость при сохранении pve-spice.vv файла в браузере Firefox
Summary: Раздражающая несправедливость при сохранении pve-spice.vv файла в браузере Fi...
Status: NEW
Alias: None
Product: Sisyphus
Classification: Development
Component: pve-manager (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Alexey Shabalin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-11-18 21:21 MSK by v.slugin
Modified: 2025-11-18 21:49 MSK (History)
5 users (show)

See Also:


Attachments
результат после отключения проверки браузеров (47.61 KB, image/png)
2025-11-18 21:21 MSK, v.slugin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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 мешавшее здесь поведение уже давным-давно исправлено.