jrchoo / pe

0 stars 0 forks source link

Nonsensical input for certain fields #5

Open jrchoo opened 9 months ago

jrchoo commented 9 months ago

Although the app specifies that linkedin profile have to follow alphanumeric characters, '-' and '_' and no whitespaces, it allows nonsensical inputs such as 'li/-'. It is unlikely that someone will have a profile with that username.

image.png

Steps:

  1. addalt 9 tg/@username 1 e/doe@gmail.com li/-
nus-se-bot commented 9 months ago

Team's Response

Thank you for raising this issue. This is an occurrence of an extreme user input which should be considered as a functionality bug as per guidelines. Moreover, we feel that users will unlikely input - into linkedin because this is as good as saying that this person has no linkedin account. Then, why would users go through the hassle of entering this command only to put a -? Of course, in our future implementation, we will consider blocking such nonsensical inputs altogether but this isn't of high priority as of our current implementation due to time constraints.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]