Skip to content

Commit 0db11e0

Browse files
authored
Added authorization requirements for posting and deleting reports (#413)
* feat:added authorization requirements for posting and deleting reports * fix: linting use
1 parent 20245ed commit 0db11e0

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

api/src/controllers/reports.ts

+4
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ const router = express.Router();
1313
* Get all reports
1414
*/
1515
router.get('/', async (req, res) => {
16+
if (!req.session.passport) return res.status(401).send('Unathenticated');
17+
if (!req.session.passport.admin) return res.status(403).send('Unauthorized');
1618
const reports = await getDocuments(COLLECTION_NAMES.REPORTS, {}); // get all reports in collection
1719

1820
res.json(reports);
@@ -34,6 +36,8 @@ router.post('/', async (req, res) => {
3436
*/
3537
router.delete('/', async (req, res) => {
3638
let status;
39+
if (!req.session.passport) return res.status(401).send('Unathenticated');
40+
if (!req.session.passport.admin) return res.status(403).send('Unauthorized');
3741
if (req.body.id) {
3842
console.log(`Deleting report ${req.body.id}`);
3943
status = await deleteDocument(COLLECTION_NAMES.REPORTS, {

0 commit comments

Comments
 (0)