Resource ownership checks in Express - Interactive Code Practice

Practice - 5 Tasks

1fill in blank

easy

Complete the code to get the user ID from the request object.

Express

const userId = req.[1].id;

Drag options to blanks, or click blank then click option'

Aparams

Buser

Cbody

Dquery

Attempts:

3 left

💡 Hint

Common Mistakes

Using req.params instead of req.user

Trying to get user ID from req.body

Using req.query for user info

The user ID is usually stored in req.user after authentication middleware runs.

2fill in blank

medium

Complete the code to compare the resource owner ID with the logged-in user ID.

Express

if (resource.ownerId.toString() === req.user.[1]) {

Drag options to blanks, or click blank then click option'

Aid

Bemail

Cusername

Drole

Attempts:

3 left

💡 Hint

Common Mistakes

Comparing resource owner ID to user email

Using username instead of user ID

Checking user role instead of ID

The id property uniquely identifies the user and is used to check ownership.

3fill in blank

hard

Fix the error in the ownership check condition.

Express

if (resource.ownerId.[1]() === req.user.id) {

Drag options to blanks, or click blank then click option'

AtoString

Bid.toString()

Cid.value

Did

Attempts:

3 left

💡 Hint

Common Mistakes

Comparing ObjectId directly to string user ID

Using incorrect property like id.value

Calling toString() on the wrong side

Since resource.ownerId is an ObjectId, convert it to string before comparing with req.user.id.

4fill in blank

hard

Fill all three blanks to correctly check ownership and send a 403 response if unauthorized.

Express

if (resource.ownerId.[1]() !== req.user.[2]) {
  return res.status(403).[3]('Forbidden');
}

Drag options to blanks, or click blank then click option'

AtoString

Bid

Csend

Djson

Attempts:

3 left

💡 Hint

Common Mistakes

Using json instead of send for simple message

Not converting ObjectId to string

Comparing wrong properties

Convert ownerId to string, compare with req.user.id, and send a 403 Forbidden response using res.status(403).send().

5fill in blank

hard

Fill all three blanks to create a middleware that checks resource ownership before proceeding.

Express

async function checkOwnership(req, res, next) {
  const resource = await Resource.findById(req.params.[1]);
  if (!resource) return res.status(404).[2]('Not found');
  if (resource.ownerId.[3]() !== req.user.id) {
    return res.status(403).send('Forbidden');
  }
  next();
}

Drag options to blanks, or click blank then click option'

Aid

Bsend

CtoString

DownerId

Attempts:

3 left

💡 Hint

Common Mistakes

Using wrong param name instead of 'id'

Using json instead of send

Not converting ObjectId to string before comparison

The middleware gets the resource by req.params.id, sends a 404 with send if not found, and compares ownerId.toString() with req.user.id to check ownership.

Practice

1. What is the main purpose of resource ownership checks in an Express app?

easy

A. To allow any user to edit any resource

B. To ensure only the owner can access or modify their resource

C. To speed up database queries

D. To log user activity for analytics

Solution

1. Step 1: Understand resource ownership

   Resource ownership means a resource belongs to a specific user.

2. Step 2: Purpose of ownership checks

   Ownership checks prevent unauthorized users from accessing or changing resources they don't own.

3. Final Answer:

   To ensure only the owner can access or modify their resource -> Option B

4. Quick Check:

   Ownership check = restrict access to owner [OK]

Hint: Ownership checks block non-owners from resource access [OK]

Common Mistakes:

• Thinking ownership checks speed up queries
• Allowing all users to edit resources
• Confusing ownership with logging

2. Which Express middleware pattern correctly checks if the logged-in user owns a resource with ID in req.params.id and owner ID in resource.ownerId?

easy

A. if (req.user.id == resource.owner) { next(); } else { res.status(401).send('Unauthorized'); }

B. if (req.user === resource.ownerId) { next(); } else { res.status(404).send('Not Found'); }

C. if (req.user.id === resource.ownerId) { next(); } else { res.status(403).send('Forbidden'); }

D. if (req.user.id !== resource.ownerId) { next(); } else { res.status(403).send('Forbidden'); }

Solution

1. Step 1: Check user ID equality

   We compare req.user.id with resource.ownerId using strict equality to confirm ownership.

2. Step 2: Respond with 403 if not owner

   If IDs don't match, respond with 403 Forbidden to block access.

3. Final Answer:

   if (req.user.id === resource.ownerId) { next(); } else { res.status(403).send('Forbidden'); } -> Option C

4. Quick Check:

   Strict equality + 403 Forbidden = correct ownership check [OK]

Hint: Use strict equality and 403 status for ownership checks [OK]

Common Mistakes:

• Using == instead of ===
• Sending wrong status codes like 404 or 401
• Comparing whole user object instead of user ID

3. Given this Express route snippet, what will happen if req.user.id is '123' and resource.ownerId is '456'?

app.delete('/items/:id', (req, res) => {
  const resource = {ownerId: '456'};
  if (req.user.id === resource.ownerId) {
    res.send('Deleted');
  } else {
    res.status(403).send('Forbidden');
  }
});

medium

A. The item will be deleted and 'Deleted' sent

B. The server will crash due to undefined resource

C. Response will be 404 Not Found

D. Response will be 403 Forbidden

Solution

1. Step 1: Compare user ID and owner ID

   Since req.user.id ('123') does not equal resource.ownerId ('456'), ownership check fails.

2. Step 2: Return 403 Forbidden

   The else block sends a 403 Forbidden response blocking deletion.

3. Final Answer:

   Response will be 403 Forbidden -> Option D

4. Quick Check:

   Non-matching IDs = 403 Forbidden [OK]

Hint: Non-owner gets 403 Forbidden response [OK]

Common Mistakes:

• Assuming deletion happens anyway
• Confusing 403 with 404
• Ignoring ownership check logic

4. Identify the bug in this ownership check middleware:

function checkOwnership(req, res, next) {
  const resource = {ownerId: '456'}; /* example */
  if (req.user.id = resource.ownerId) {
    next();
  } else {
    res.status(403).send('Forbidden');
  }
}

medium

A. Using assignment (=) instead of comparison (===) in the if condition

B. Missing call to next() in else block

C. Incorrect status code; should be 404 instead of 403

D. resource.ownerId is undefined

Solution

1. Step 1: Check the if condition syntax

   The condition uses single equals (=), which assigns instead of compares, causing a bug.

2. Step 2: Correct comparison operator

   It should use strict equality (===) to compare req.user.id and resource.ownerId.

3. Final Answer:

   Using assignment (=) instead of comparison (===) in the if condition -> Option A

4. Quick Check:

   Assignment in if condition = bug [OK]

Hint: Use === for comparison, not = assignment [OK]

Common Mistakes:

• Confusing = with === in conditions
• Thinking next() needed in else block
• Wrong status code for forbidden access

5. You want to protect a route so only the owner of a blog post can edit it. The post's owner ID is stored in post.ownerId. Which Express middleware correctly implements this ownership check and returns 403 if the user is not the owner?

hard

A. app.put('/posts/:id', (req, res, next) => { if (req.user.id === post.ownerId) next(); else res.status(403).send('Forbidden'); }, (req, res) => { res.send('Post updated'); });

B. app.put('/posts/:id', (req, res) => { if (req.user.id !== post.ownerId) res.status(403).send('Forbidden'); else res.send('Post updated'); });

C. app.put('/posts/:id', (req, res, next) => { if (req.user.id == post.ownerId) next(); else res.status(404).send('Not Found'); }, (req, res) => { res.send('Post updated'); });

D. app.put('/posts/:id', (req, res) => { if (req.user.id === post.ownerId) res.send('Post updated'); else res.status(401).send('Unauthorized'); });

Solution

1. Step 1: Use middleware to check ownership before update

   Middleware checks if req.user.id matches post.ownerId and calls next() if true.

2. Step 2: Return 403 Forbidden if not owner

   If IDs don't match, respond with 403 to block unauthorized edits.

3. Final Answer:

   app.put('/posts/:id', (req, res, next) => { if (req.user.id === post.ownerId) next(); else res.status(403).send('Forbidden'); }, (req, res) => { res.send('Post updated'); }); -> Option A

4. Quick Check:

   Middleware + strict equality + 403 Forbidden = correct pattern [OK]

Hint: Use middleware with strict check and 403 response [OK]

Common Mistakes:

• Using == instead of ===
• Sending wrong status codes like 404 or 401
• Not using middleware pattern for ownership check