Backup: не хранить расшифрованные секреты в модели предпросмотра

Đang mở
  • Có gì không ổn?
#396 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
35/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
csharp
Lĩnh vực
desktop, security

Hướng nghiên cứu

Start by reading BackupArchive.Decrypt and SettingsViewModel._pendingBackup, then trace the preview, apply, cancel, and rollback flows. The issue names no test files, so locate the synthetic backup and restore tests and the benchmark entry point. Done means preview metadata contains no secret values, apply preserves restore and rollback behavior, and session buffers are released across the listed lifecycle paths.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Проблема

BackupArchive.Decrypt создаёт полный BackupData, включая строки UUID, паролей, приватных ключей и исходных конфигураций. SettingsViewModel._pendingBackup удерживает этот граф весь период предпросмотра, хотя интерфейсу для списка изменений нужны идентификаторы, имена и связи. CancelBackup снимает ссылку, но не обнуляет неизменяемые строки. Очистка основного plaintext-массива в Decrypt эти строки не очищает.

Решение

Ввести управляемую C#-сессию импорта с явным жизненным циклом. После проверки аутентичности строить только несекретные метаданные предпросмотра, не материализуя значения секретов в string. Полный BackupData создавать непосредственно перед применением и не сохранять в ViewModel после завершения операции.

Для сохранения сценария с однократным вводом пароля сессия может владеть зашифрованными байтами архива и производным ключом в отдельном очищаемом буфере; сам пароль не нужен после открытия. Это сокращает число и срок жизни plaintext-копий, но не защищает от атакующего с доступом к памяти процесса: ключ позволяет расшифровать архив. Не заявлять гарантированное стирание всех секретов или защиту от дампов.

Готово, когда

  • Предпросмотр сохраняет список добавлений/замен/удалений и переключение Merge/Replace, не возвращая значения секретов в свои DTO или ViewModel.
  • Неверный пароль и повреждённый архив отклоняются до показа предпросмотра; полная проверка содержимого завершается до любых изменений профилей, настроек и секретов.
  • Применение не требует повторного ввода пароля; сессия эксклюзивно владеет ключом и временными plaintext-буферами и очищает их при освобождении.
  • Отмена, выбор другого файла, уход с экрана, успешное применение и завершение приложения освобождают сессию; повторный Dispose безопасен. Поведение после ошибки применения явно определено и не теряет управление ресурсами.
  • Синтетические тесты проверяют жизненный цикл, отсутствие секретов в метаданных, очистку контролируемых буферов и сохранение семантики восстановления/rollback.
  • Бенчмарк показывает изменение аллокаций на предпросмотре отдельно от стоимости применения и повторной расшифровки; нет обещания выигрыша скорости без измерений.

Связано с #18, #344 и PR #375. Решение остаётся на C#, формат архива и сетевое поведение не меняются. Нативная библиотека и Rust/Zig не входят в задачу. Все тесты — BYDLO_SAFE_DEV=1, фейки; UI-проверки только с явным --mock.

Ngôn ngữ chính
C#
Star
5
Fork
1
Merge trung bình
30 phút
Pull request đã merge (30 ngày)
171

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của endercodezz/BYDLO

Tất cả issue của endercodezz/BYDLO

Issue tương tự

Thêm issue về C#

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.