Add a "don't ask me anymore" option to the interactive mode #128

Activity

1. KevinHock

   added

   good first issue

   help wanted

   on May 5, 2018
2. prithvisingh18 commented on Jun 11, 2018

   

   prithvisingh18

   on Jun 11, 2018

   More actions

   Hi, can I try to solve this issue ?

   👍React with 👍1Reacted by KevinHock
3. KevinHock commented on Jun 11, 2018

   

   KevinHock

   on Jun 11, 2018

   CollaboratorAuthor

   More actions

   Hi prithvisingh, definitely 👍🙏

   I'll try to make the issue description etc. better later today, but for understanding the code a bit more see this pr #81 :)

   👍React with 👍1Reacted by Prithvi Singh
4. KevinHock commented on Jun 12, 2018

   

   KevinHock

   on Jun 12, 2018

   Last edited by KevinHock

   CollaboratorAuthor

   More actions

   Hi @prithvisingh18
   So in vulnerabilities.py

   We can change it here when the user types stop
   https://github.com/python-security/pyt/blob/master/pyt/vulnerabilities/vulnerabilities.py#L304-L310
   e.g. change it to Is the return value of {} with tainted argument "{}" vulnerable? (Y/n/stop)
   and check for if user_says.startswith('s'):

   This seems like it can be solved in 1 of 3 ways

   • make ui_mode an object, not an int (so that we can change it by reference)
   • use a global variable everywhere we currently use it (so that we can set the global to the variable)
   • raise an exception, catch the exception and then proceed with a different value

   These are all kind of ugly, what do you think is nice? :)
   I kind of think making it an object would be the best way, b/c trying to catch an exception in the loop, and still keeping what the user previously entered via y and n will lead to ugly code.

   👍React with 👍1Reacted by Prithvi Singh
5. KevinHock

   closed this as completedin #190on Jan 30, 2019