search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Data is not encrypted #4782

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

One NFR states: The data stored within the app should be encrypted for security purposes (to prevent the raw data being read by external parties).

However, I seem to be able to read the patients' details (such as medical conditions, blood type etc.) directly from the addressbook JSON file. This could be confusing to new developers.

[original: nus-cs2103-AY2122S1/pe-interim#4736] [original labels: severity.Medium type.DocumentationBug]

simonjulianl commented 3 years ago

Team's Response

Thank you for the response :)

image.png

As mentioned in the CS2103T website, unfulfilled NFR should be of low severity.

Duplicate status (if any):

--