fix: cache middleware now checks response status before storing (fixes #1197)

[vaishnavi003-svg]

Problem

API cache was saving error responses (401, 403, 500) and serving them as valid data to users.

Fix

Added status code check before caching - now only success responses (200-299) are cached.

Changes

• middleware/verifyToken.js - Added status validation
• middleware/cacheMiddleware.js - Only caches 2xx responses
• app.js - Updated middleware order

Test Results

Response	Before	After
200 Success	✅ Cached	✅ Cached
401/403 Error	❌ Cached	✅ Not Cached
500 Error	❌ Cached	✅ Not Cached

Code Change

// Before - cached everything
cache.set(key, data);

// After - only cache success
if (res.statusCode >= 200 && res.statusCode < 300) {
    cache.set(key, data);
} else {
    cache.del(key); // clear error cache
}

[github-actions]

Thank you @, for creating the PR and contributing to our UltimateHealth project 💗.
Our team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀