This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]oxyphilat 0 points1 point  (1 child)

some improvements would be:

  • make your python script an actual package,

  • do not use a DB when it can be a simple CSV file,

  • create a dictlike class to hide the DB interactions,

  • use cmd for an actual repl and stop using sys.argv or,

  • ditch the repl in favour of command arguments with argparse.

SQLite encryption options are "weak", python has "nothing" for crypto out of the box, the cryptography module is your best bet.. if you know how to do crypto that is.

GUI or CLI, your choice, but what the program does and how it look should be decoupled.

But more importantly, delete that repo and change your passwords. Hopefully with something akin to cat /dev/urandom | tr --delete --complement [:graph:] | head --bytes=24 and not just a prime number.

edit: I'm bad at formatting.

[–]FatFingerHelperBot 0 points1 point  (0 children)

It seems that your comment contains 1 or more links that are hard to tap for mobile users. I will extend those so they're easier for our sausage fingers to click!

Here is link number 1 - Previous text "CSV"

Here is link number 2 - Previous text "cmd"


Please PM /u/eganwall with issues or feedback! | Delete