Прерванный экспорт поверх существующего файла уничтожает старую резервную копию

已关闭
#334 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
1-2 天
新手友好度
54/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
csharp
领域
backend, desktop

调研方向

Start with SettingsViewModel.ExportBackup in SettingsViewModel.Backup.cs (lines 67–81) and inspect how the destination stream is opened and written. Check the backup export tests and the fake storage stream behavior, if present. Done means an interrupted overwrite leaves the previous backup readable, while providers without safe replacement offer saving to a new file.

由索引模型根据 Issue 内容生成。

描述

bug

Приоритет P1 · milestone 0.4.1

После подтверждения замены файла экспорт может оставить пустую или неполную копию, если запись прервётся после открытия файла. Пользователь теряет предыдущую рабочую резервную копию.

Воспроизведение с фейковым хранилищем

  1. Сохранить корректную .bydlobackup.
  2. Экспортировать новую копию в тот же файл с подтверждением перезаписи.
  3. Подменить поток сохранения на поток, который успешно выполняет SetLength(0), а на записи выбрасывает IOException. Такой порядок соответствует сбою диска или облачного провайдера при записи.
  4. Команда покажет ошибку. Предыдущий файл уже обрезан и не проходит проверку при импорте.

Причина и доказательство: SettingsViewModel.ExportBackup (SettingsViewModel.Backup.cs:67–81) напрямую обнуляет целевой поток и записывает туда результат. Обработчик ошибки показывает сообщение без восстановления старых байтов. Подтверждено порядком операций, на пользовательских файлах сбои не симулировались.

Ожидаемо: предыдущий файл остаётся читаемым до полного завершения экспорта новой копии.

Направление исправления: для локального файла записывать и сбрасывать зашифрованный временный файл в той же папке, затем атомарно заменять назначение. Для провайдеров без безопасной замены предложить сохранить новый файл.

Аудит ffa65648867139f9e45c15f4b635a83c1ff52c1b.

主要语言
C#
星标
5
派生
1
平均合并
30 分钟
30 天内合并 PR
171

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

endercodezz/BYDLO 的其他 Issue

查看 endercodezz/BYDLO 的全部 Issue

相似的 Issue

更多 C# Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。