From edd17cc121d221da1267c1292a3232254f3181f5 Mon Sep 17 00:00:00 2001
From: Ajay Dhangar <ajaydhangar49@gmail.com>
Date: Wed, 6 May 2026 08:48:38 +0530
Subject: [PATCH] solve issues

---
 .../full-stack/backend/error-handling.mdx     | 12 +++++++++
 .../full-stack/backend/express.mdx            | 26 +++++++++++++++++++
 .../full-stack/backend/mvc-pattern.mdx        | 15 +++++++++++
 .../full-stack/backend/nodejs-basics.mdx      | 14 ++++++++++
 .../full-stack/backend/routing.mdx            | 10 +++----
 5 files changed, 72 insertions(+), 5 deletions(-)

diff --git a/absolute-beginners/full-stack/backend/error-handling.mdx b/absolute-beginners/full-stack/backend/error-handling.mdx
index 8f3e910..6dd4b25 100644
--- a/absolute-beginners/full-stack/backend/error-handling.mdx
+++ b/absolute-beginners/full-stack/backend/error-handling.mdx
@@ -43,6 +43,18 @@ app.use((err, req, res, next) => {
 });
 ```
 
+:::info Security Tip
+Exposing the error message directly to the client can leak sensitive information about the server's internal state. It is safer to only include detailed error information when the application is running in a development environment.
+
+```javascript title="server.js"
+  res.status(500).json({  
+    success: false,  
+    message: "Internal Server Error",  
+    error: process.env.NODE_ENV === 'development' ? err.message : {}  
+  });  
+```
+:::
+
 ## 3. Throwing Custom Errors
 
 Sometimes, the code isn't "broken," but the user did something wrong (like entering the wrong password). In these cases, you can "throw" your own error to trigger the catch block.
diff --git a/absolute-beginners/full-stack/backend/express.mdx b/absolute-beginners/full-stack/backend/express.mdx
index 805a1d2..c075db1 100644
--- a/absolute-beginners/full-stack/backend/express.mdx
+++ b/absolute-beginners/full-stack/backend/express.mdx
@@ -93,6 +93,32 @@ npm install -g nodemon
 nodemon server.js
 ```
 
+:::info
+Installing packages globally with `-g` is generally discouraged for project-specific tools. It is better to install nodemon as a development dependency to ensure that all developers working on the project use the same version.
+
+```bash
+npm install --save-dev nodemon
+```
+
+Then, you can add a script to your `package.json`:
+
+```json title="package.json"
+{
+  "scripts": {
+    "dev": "nodemon server.js"
+  }
+}
+```
+
+Now you can start your server with:
+
+```bash
+npm run dev
+```
+
+This way, everyone on the project will have the same version of nodemon, and you won't run into issues if someone doesn't have it installed globally.
+:::
+
 ## Practice: The Dynamic Greeter
 
 Let's use **URL Parameters** to greet users by name!
diff --git a/absolute-beginners/full-stack/backend/mvc-pattern.mdx b/absolute-beginners/full-stack/backend/mvc-pattern.mdx
index e5c8611..3b08753 100644
--- a/absolute-beginners/full-stack/backend/mvc-pattern.mdx
+++ b/absolute-beginners/full-stack/backend/mvc-pattern.mdx
@@ -88,6 +88,21 @@ exports.getProfile = async (req, res) => {
 };
 ```
 
+:::info
+The `async` controller function lacks error handling. If the database call fails, the promise will reject without being caught, which can lead to unhandled promise rejections. It is best practice to wrap the logic in a `try...catch` block and pass the error to the `next` middleware.
+
+```javascript title="userController.js"
+exports.getProfile = async (req, res, next) => {  
+    try {  
+        const userData = await User.findById(req.params.id);  
+        res.json(userData);  
+    } catch (error) {  
+        next(error);  
+    }  
+};
+```
+:::
+
 **3. The Model (`/models/userModel.js`):** The definition of the data.
 
 ```javascript title="userModel.js"
diff --git a/absolute-beginners/full-stack/backend/nodejs-basics.mdx b/absolute-beginners/full-stack/backend/nodejs-basics.mdx
index 8b8eaf1..fa3d7bd 100644
--- a/absolute-beginners/full-stack/backend/nodejs-basics.mdx
+++ b/absolute-beginners/full-stack/backend/nodejs-basics.mdx
@@ -92,6 +92,20 @@ fs.readFile('note.txt', 'utf8', (err, data) => {
 });
 ```
 
+:::info Error Handling
+In the above example, if `note.txt` does not exist, the `err` object will contain the error details. Throwing an error inside an asynchronous callback will cause the Node.js process to crash. Instead of throwing, you should log the error or handle it gracefully to keep the server running.
+
+```javascript title="readFile.js"
+fs.readFile('note.txt', 'utf8', (err, data) => {  
+  if (err) {  
+    console.error("Error reading file:", err);  
+    return;  
+  }  
+  console.log("File content:", data);  
+});
+```
+:::
+
 ## Practice: The Greeting Machine
 
 1.  Create a file named `greet.js`.
diff --git a/absolute-beginners/full-stack/backend/routing.mdx b/absolute-beginners/full-stack/backend/routing.mdx
index ff36bc4..53ac3eb 100644
--- a/absolute-beginners/full-stack/backend/routing.mdx
+++ b/absolute-beginners/full-stack/backend/routing.mdx
@@ -118,11 +118,11 @@ router.get('/all', (req, res) => {
 router.get('/:id', (req, res) => {
   const bookId = req.params.id;
   // In a real app, you'd fetch the book from the database
-  if (bookId == 1) {
-    res.json({ id: 1, title: "The Great Gatsby" });
-  } else if (bookId == 2) {
-    res.json({ id: 2, title: "To Kill a Mockingbird" });
-  } else {
+  if (bookId === "1") {  
+    res.json({ id: 1, title: "The Great Gatsby" });  
+  } else if (bookId === "2") {  
+    res.json({ id: 2, title: "To Kill a Mockingbird" });  
+  } else {  
     res.status(404).send("Book not found!");
   }
 });