realroyxu / CITS3403-MurderMystery

A repo for UWA CITS3403's group project
GNU General Public License v3.0
1 stars 0 forks source link

Security Issue: moderate priority #76

Open realroyxu opened 4 months ago

realroyxu commented 4 months ago

We only use session['userid'] to validate at backend. This is well know vulnerability, well ,it'll get us mark deducted. If we have enough time we can think about how do we change it to a more safer way, if not, not too bad since at least logic is running correct.

realroyxu commented 4 months ago

Okay so I tested what if I moderate userid at cookie/session and try to stole identity, that's not working. So it's still relatively safe at status quo, hopefully.