Open fatclarence opened 4 years ago
Thanks for the report.
The User Guide defines a valid URL as follows:
(This description can also be viewed in the application when an invalid URL is entered.)
Our definition of a valid URL is based on the URI generic syntax. We chose to make our URL validation more lenient by allowing some invalid URLs instead of potentially rejecting valid URLs (which would negatively impact the app's usefulness). Validating URLs perfectly would require the app to actually check whether a web page exists at the given address, which greatly diminishes performance; the check might also fail in some cases (e.g. too many redirects).
We acknowledge that the validation is imperfect. However, because it is impossible to validate URLs perfectly, we are marking this bug as out of scope for this CS2103T project. In other words, we believe that this is a valid issue but not one that the team should be penalized for.
Team chose [response.NotInScope
]
Reason for disagreement: [replace this with your reason]
Team chose [severity.Low
].
Originally [severity.Medium
].
Reason for disagreement: [replace this with your reason]
Summary: An invalid URL can be added as a bookmark. Eg. "https://monkey" "https://google"
Steps to reproduce:
Expected results: Throw invalid url error because the url is invalid.
Actual results: Result displays show success message: "New bookmark added: monkeys URL: https://google Remark: Module planner Folder: ROOT Tags: [school][timetable]"
Screenshot of issue: