ComputerScienceHouse / proxstar

VM self-service tool for Computer Science House that integrates with Proxmox and STARRS
https://csh.rit.edu/projects/2018/04/12/proxstar.html
MIT License
23 stars 21 forks source link

Remove SSH functionality from Proxstar #158

Closed WillNilges closed 2 years ago

WillNilges commented 2 years ago

Now that VNC is over wss://, we don't need it.

WillNilges commented 2 years ago

In regards to the TODOs, I can use Redis to store the targets file data :0

WillNilges commented 2 years ago
diff --git a/proxstar/tasks.py b/proxstar/tasks.py
index a94ff70..0bf006f 100644
--- a/proxstar/tasks.py
+++ b/proxstar/tasks.py
@@ -151,7 +151,7 @@ def process_expiring_vms_task():
                             vm.name, vm.id
                         )
                     )
-                    vmid = vm['vmid'] 
+                    vmid = vm['vmid']
                     vnc_token_key = f'vnc_token|{vmid}'
                     vnc_token = redis_conn.get(vnc_token_key).decode('utf-8')
                     delete_vnc_target(token=vnc_token)

Thank you black, very cool.

WillNilges commented 2 years ago

bro smh

WillNilges commented 2 years ago

gonna rebase this a little because I want to be able to bisect this if the tasks break.