ColoradoSchoolOfMines / acm-site

Source code for acm.mines.edu
https://acm.mines.edu
GNU General Public License v3.0
2 stars 0 forks source link

Testing: Extensive vulnerability checks #15

Closed ezrichards closed 8 months ago

ezrichards commented 1 year ago

We need to extensively test the site for vulnerabilities.

I've looked mainly into SQL injection potentially so far, and we should continue to look into this. A lot of our queries are "static" and have no user fields going directly into them, but some do.

In order to feel comfortable about vulnerabilities on the final product, we should investigate/verify:

These may not necessarily have PRs associated with them, but we should have discussions about all of these things if possible.

ezrichards commented 1 year ago

We should also ensure that admin sessions are deleted when admins are edited.

This is pretty hard to test locally (as we only have access to one mines account), so we can test in prod before we release.

ezrichards commented 8 months ago

We might want to ask OreSec to penetration test too