chore(backend.js): better error handling
This commit is contained in:
parent
fe2ab59aaf
commit
8c2ebe8dfa
@ -129,7 +129,7 @@ const activeSessions = new Map();
|
|||||||
// Utility function to create a timeout promise
|
// Utility function to create a timeout promise
|
||||||
const timeoutPromise = (ms) => {
|
const timeoutPromise = (ms) => {
|
||||||
return new Promise((_, reject) => {
|
return new Promise((_, reject) => {
|
||||||
setTimeout(() => reject(new Error(`Request timed out after ${ms}ms`)), ms);
|
setTimeout(() => reject(new ApiTimeoutError(`Request timed out after ${ms}ms`)), ms);
|
||||||
});
|
});
|
||||||
};
|
};
|
||||||
|
|
||||||
@ -153,35 +153,78 @@ const ensureLogin = async (ssoToken) => {
|
|||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
// Helper function to handle API errors
|
||||||
|
class ApiTimeoutError extends Error {
|
||||||
|
constructor(message = 'API request timed out') {
|
||||||
|
super(message);
|
||||||
|
this.name = 'ApiTimeoutError';
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
class ApiAuthError extends Error {
|
||||||
|
constructor(message = 'Authentication failed') {
|
||||||
|
super(message);
|
||||||
|
this.name = 'ApiAuthError';
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
class ApiDataError extends Error {
|
||||||
|
constructor(message = 'Data processing error') {
|
||||||
|
super(message);
|
||||||
|
this.name = 'ApiDataError';
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// Helper function to handle API errors
|
// Helper function to handle API errors
|
||||||
const handleApiError = (error, res) => {
|
const handleApiError = (error, res) => {
|
||||||
logger.error('API Error:', error);
|
logger.error('API Error:', error);
|
||||||
logger.error(`Error Stack: ${error.stack}`);
|
logger.error(`Error Stack: ${error.stack}`);
|
||||||
logger.error(`Error Time: ${global.Utils.toIsoString(new Date())}`);
|
logger.error(`Error Time: ${global.Utils.toIsoString(new Date())}`);
|
||||||
|
|
||||||
// Try to extract more useful information from the error
|
// Map error types to appropriate responses
|
||||||
let errorMessage = error.message || 'Unknown API error';
|
const errorResponses = {
|
||||||
let errorName = error.name || 'ApiError';
|
'ApiTimeoutError': {
|
||||||
|
status: 200,
|
||||||
// Handle the specific JSON parsing error
|
body: {
|
||||||
if (errorName === 'SyntaxError' && errorMessage.includes('JSON')) {
|
|
||||||
logger.debug('JSON parsing error detected');
|
|
||||||
return res.status(200).json({
|
|
||||||
status: 'error',
|
status: 'error',
|
||||||
message:
|
message: 'The request timed out. Please try again.',
|
||||||
'Failed to parse API response. This usually means the SSO token is invalid or expired.',
|
error_type: 'timeout',
|
||||||
error_type: 'InvalidResponseError',
|
timestamp: global.Utils.toIsoString(new Date())
|
||||||
timestamp: global.Utils.toIsoString(new Date()),
|
|
||||||
});
|
|
||||||
}
|
}
|
||||||
|
},
|
||||||
// Send a more graceful response
|
'ApiAuthError': {
|
||||||
return res.status(200).json({
|
status: 200,
|
||||||
|
body: {
|
||||||
status: 'error',
|
status: 'error',
|
||||||
message: errorMessage,
|
message: 'Authentication failed. Please check your SSO token.',
|
||||||
error_type: errorName,
|
error_type: 'auth_failure',
|
||||||
timestamp: global.Utils.toIsoString(new Date()),
|
timestamp: global.Utils.toIsoString(new Date())
|
||||||
});
|
}
|
||||||
|
},
|
||||||
|
'SyntaxError': {
|
||||||
|
status: 200,
|
||||||
|
body: {
|
||||||
|
status: 'error',
|
||||||
|
message: 'Failed to parse API response. This usually means the SSO token is invalid or expired.',
|
||||||
|
error_type: 'InvalidResponseError',
|
||||||
|
timestamp: global.Utils.toIsoString(new Date())
|
||||||
|
}
|
||||||
|
},
|
||||||
|
'default': {
|
||||||
|
status: 200,
|
||||||
|
body: {
|
||||||
|
status: 'error',
|
||||||
|
message: error.message || 'An unknown error occurred',
|
||||||
|
error_type: error.name || 'UnknownError',
|
||||||
|
timestamp: global.Utils.toIsoString(new Date())
|
||||||
|
}
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
// Get the appropriate response or use default
|
||||||
|
const response = errorResponses[error.name] || errorResponses.default;
|
||||||
|
|
||||||
|
return res.status(response.status).json(response.body);
|
||||||
};
|
};
|
||||||
|
|
||||||
// Helper function to remove sensitive data from headers
|
// Helper function to remove sensitive data from headers
|
||||||
|
Loading…
x
Reference in New Issue
Block a user