fix(cursor): address PR #528 review feedback

- Replace non-null assertions with guard clauses in cursor-routes.ts
- Add port range validation (1-65535) in cursor-settings-routes.ts
This commit is contained in:
Tam Nhu Tran
2026-02-12 03:58:23 +07:00
parent 55bc53152c
commit d58e98815b
2 changed files with 13 additions and 7 deletions
+4 -4
View File
@@ -121,15 +121,15 @@ router.post('/auth/auto-detect', async (_req: Request, res: Response): Promise<v
try {
const result = autoDetectTokens();
if (!result.found) {
res.status(404).json({ error: result.error });
if (!result.found || !result.accessToken || !result.machineId) {
res.status(404).json({ error: result.error ?? 'Token not found' });
return;
}
// Save credentials
saveCredentials({
accessToken: result.accessToken!,
machineId: result.machineId!,
accessToken: result.accessToken,
machineId: result.machineId,
authMethod: 'auto-detect',
importedAt: new Date().toISOString(),
});
@@ -34,9 +34,15 @@ router.put('/', (req: Request, res: Response): void => {
// Validate input types
if (updates && typeof updates === 'object') {
if ('port' in updates && typeof updates.port !== 'number') {
res.status(400).json({ error: 'port must be a number' });
return;
if ('port' in updates) {
if (typeof updates.port !== 'number' || !Number.isInteger(updates.port)) {
res.status(400).json({ error: 'port must be an integer' });
return;
}
if (updates.port < 1 || updates.port > 65535) {
res.status(400).json({ error: 'port must be between 1 and 65535' });
return;
}
}
if ('auto_start' in updates && typeof updates.auto_start !== 'boolean') {
res.status(400).json({ error: 'auto_start must be a boolean' });