diff options
| author | schneefux <schneefux+commit@schneefux.xyz> | 2016-06-07 18:48:09 +0200 |
|---|---|---|
| committer | schneefux <schneefux+commit@schneefux.xyz> | 2016-06-07 18:48:09 +0200 |
| commit | b9aad5580e012978c5eb9bb35e57def4b01d5927 (patch) | |
| tree | 244e0388e298ea72064132115c66f8d43423cf40 | |
| parent | fad28e744605c645c2d98e7a885e49dc498c974f (diff) | |
| download | jokevote-b9aad5580e012978c5eb9bb35e57def4b01d5927.tar.gz jokevote-b9aad5580e012978c5eb9bb35e57def4b01d5927.zip | |
throw 403 for unauthorized actions
| -rw-r--r-- | app.py | 15 |
1 files changed, 9 insertions, 6 deletions
@@ -319,8 +319,9 @@ def edit(): text = request.form['text'] page = request.form['redirpage'] objectId = int(request.form['id']) - if objectId in db().getUserJokes(userid()): - db().updateJoke(text, objectId) + if objectId not in db().getUserJokes(userid()): + abort(403) + db().updateJoke(text, objectId) return redirect('/page/' + page) @app.route('/upvote', methods=['POST']) @@ -354,16 +355,18 @@ def report(): def delete(): objectId = int(request.form['id']) page = request.form['redirpage'] - if objectId in db().getUserJokes(userid()): - db().removeJoke(objectId, userid()) + if objectId not in db().getUserJokes(userid()): + abort(403) + db().removeJoke(objectId, userid()) return redirect('/page/' + page) @app.route('/undelete', methods=['POST']) def undelete(): objectId = int(request.form['id']) page = request.form['redirpage'] - if objectId in db().getUserJokes(userid()): - db().unvoteJoke(objectId, userid()) + if objectId not in db().getUserJokes(userid()): + abort(403) + db().unvoteJoke(objectId, userid()) return redirect('/page/' + page) @app.route('/login', methods=['POST']) |
