search

raynerljm / pe

0 stars 0 forks source link

System does not accept uppercase letters in the URL's second-level domain #6

Open raynerljm opened 2 years ago

raynerljm commented 2 years ago

Bug: System does not accept uppercase letters in the URL's second-level domain

image.png

Description: The application does not allow me to enter uppercase letters in the second-level domains (i.e. M1 in https://www.M1.com). While it is true that it is always lowercase, there are occasions when users type in the URL themselves and they might uppercase it by habit (eg. typing www.Google.com). Browsers automatically make the letters lowercase so it will still be functional when using the open command.

Steps to reproduce: Enter command add n/M1 u/https://www.M1.com d/16-04-2022 1600 t/tag3 dur/2 m/1 r/Y t/tag1 t/tag2

Expected outcome: Either: (1) The system should accept it; or (2) The system should provide a descriptive message saying that the uppercase letter is not permitted

Actual outcome: The error URLs should be a valid link, and it should not be blank is given which does not guide users on how to fix it.

soc-pe-bot commented 2 years ago

Team's Response

Thank you for raising this issue.

We note that the domain name is treated as case-insensitive by browsers. However, browsers usually will convert the domain name to lowercase, and since the main use case of our app is to copy the full meeting link into the command box instead of typing it out manually, this is out of scope for our application at the current version. We may consider updating the regex to allow capital letters in the domain name in the future.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]