nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

When data file is tampered with, system may not be able to start up / clean up the corrupted file #411

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

As written in the user guide, the user is not expected to tamper with the data files (hence severity low). However, the system should still be able to recover from it e.g. by starting fresh. Instead, the system is rendered unusable.

image.png


[original: nus-cs2113-AY2223S1/pe-interim#403] [original labels: severity.Low type.FunctionalityBug]

cheehengk commented 1 year ago

Team's Response

The dev acknowledges that there is indeed an issue with the functioning of the program with corrupted (tampered) data file. However, the dev team believes this is not in scope "as rectifying it is less important than the work that has been done already". Furthermore, as seen in the screenshot below, the UG clearly indicates that it is out of the scope of the product to fully function with illegally tampered files. Users should not be editing the data file in the first place. Data corruption protocol only seeks to be a safety net for unexpected corruption such as during crashes.This is further emphasised in the DG (second screenshot).

Screenshot 2022-11-14 at 12.48.44 AM.png

Screenshot 2022-11-14 at 12.48.35 AM.png

Duplicate status (if any):

--