search

JIceberg / grouch

A registration aid for Oscar. Simple, effective, and customizable. No login necessary.
GNU General Public License v3.0
32 stars 11 forks source link

Add Phone/Email Notification Option #2

Open JIceberg opened 3 years ago

JIceberg commented 3 years ago

Easy peasy command argument addition. Just make it the second argument (i.e. sys.argv[1]) and then sys.argv[2:] is the list of CRNs.

JIceberg commented 3 years ago

One of the problems with this it seems is that I have to make a throwaway account. Which is fine, but anyone can just do whatever with it. May need to add a disclaimer or whatever about that, but I feel like it shouldn't matter. It's a throwaway, so if it gets logged into, they can't do much except see emails that have been sent and maybe send a few emails, but I think it's fine(?) It's a terrible idea in practice though and not very smart for a simple script system.

JIceberg commented 3 years ago

Looks like the phone number is also very insecure. For this project I feel like this isn't the wisest thing to do. I will leave the issue open but will likely only do email notifications and add a disclaimer.

JIceberg commented 3 years ago

Another option which may be a bit more viable in the future if I decide to actually run this thing w/ a central server is set up a central server that takes post requests from the script and then sends emails/phone messages from that.