Merge pull request #1 from Mingy123/patch-1

Disallow NaN in transactions
appventure-nush · Oct 29, 2024 · 14a9230 · 14a9230
2 parents 1c90118 + 309df87
commit 14a9230
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/backend/src/api/student.ts b/backend/src/api/student.ts
@@ -33,6 +33,9 @@ router.post("/createTransaction", async (req, res) => {
     if (amountToDP.lte(0)) {
         return res.status(400).json({ message: "Amount must be positive" })
     }
+    if (!amountToDP.isFinite()) {
+        return res.status(400).json({ message: "Invalid Amount" })
+    }
 
     const transId = getRandom()