paul
1773ed5f95
Mirror to GitHub / mirror (push) Successful in 26s
Test and Lint / backend-test (push) Successful in 1m11s
continuous-integration/drone/push Build is passing
Test and Lint / frontend-test (push) Successful in 2m28s
Version and Release / version-bump (push) Successful in 32s
Version and Release / trigger-drone (push) Has been skipped
Original: feat: enhance security logging and ensure rate limit blocks are properly tracked - Add comprehensive logging for rate limit blocks with full request details - IP address (with proper proxy detection), user agent, headers, timestamps - Rate limit info (current count, limit, remaining, reset time) - Separate tracking for auth vs general endpoints - Enhance authentication failure logging - JWT validation failures with detailed error info - Admin auth attempts without token - Failed token validation with user context - All events include IP, path, method, user agent - Improve Winston logger configuration for production - Add automatic log rotation (10MB errors, 50MB combined) - Create separate security.log for auth/rate limit events - Ensure logs directory exists automatically - Add structured JSON format for log aggregation - Support container logging with LOG_TO_CONSOLE env var - Create comprehensive documentation - Security logging guide with examples - Monitoring recommendations - Configuration reference - Add test script to verify logging functionality All rate limit settings remain configurable via admin panel: - Window duration, max requests, auth limits - Skip authenticated requests option - Public endpoints only option 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
83 lines
2.5 KiB
JavaScript
83 lines
2.5 KiB
JavaScript
const { formatBoolean, isPostgreSQL, addDays, formatDateForDB, insertAndGetId } = require('../utils/dbCompat');
|
|
|
|
describe('Database Compatibility', () => {
|
|
// Save original env
|
|
const originalEnv = process.env.DATABASE_CLIENT;
|
|
|
|
afterEach(() => {
|
|
// Restore original env after each test
|
|
if (originalEnv) {
|
|
process.env.DATABASE_CLIENT = originalEnv;
|
|
} else {
|
|
delete process.env.DATABASE_CLIENT;
|
|
}
|
|
});
|
|
|
|
describe('formatBoolean', () => {
|
|
test('should format boolean values correctly', () => {
|
|
// Mock for SQLite
|
|
process.env.DATABASE_CLIENT = 'sqlite3';
|
|
expect(formatBoolean(true)).toBe(1);
|
|
expect(formatBoolean(false)).toBe(0);
|
|
|
|
// Mock for PostgreSQL
|
|
process.env.DATABASE_CLIENT = 'pg';
|
|
expect(formatBoolean(true)).toBe(true);
|
|
expect(formatBoolean(false)).toBe(false);
|
|
|
|
// Default (no env var) should be SQLite
|
|
delete process.env.DATABASE_CLIENT;
|
|
expect(formatBoolean(true)).toBe(1);
|
|
expect(formatBoolean(false)).toBe(0);
|
|
});
|
|
});
|
|
|
|
describe('isPostgreSQL', () => {
|
|
test('should detect PostgreSQL correctly', () => {
|
|
process.env.DATABASE_CLIENT = 'pg';
|
|
expect(isPostgreSQL()).toBe(true);
|
|
|
|
process.env.DATABASE_CLIENT = 'sqlite3';
|
|
expect(isPostgreSQL()).toBe(false);
|
|
|
|
delete process.env.DATABASE_CLIENT;
|
|
expect(isPostgreSQL()).toBe(false); // Default to SQLite
|
|
});
|
|
});
|
|
|
|
describe('formatDateForDB', () => {
|
|
test('should format dates as ISO strings', () => {
|
|
const date = new Date('2024-01-15T10:30:00Z');
|
|
expect(formatDateForDB(date)).toBe('2024-01-15T10:30:00.000Z');
|
|
});
|
|
});
|
|
|
|
describe('addDays', () => {
|
|
test('should add days correctly', () => {
|
|
const date = new Date('2024-01-15');
|
|
const result = addDays(date, 30);
|
|
expect(result.toISOString().split('T')[0]).toBe('2024-02-14');
|
|
|
|
const negativeResult = addDays(date, -7);
|
|
expect(negativeResult.toISOString().split('T')[0]).toBe('2024-01-08');
|
|
});
|
|
});
|
|
|
|
describe('insertAndGetId', () => {
|
|
test('should handle PostgreSQL result format', async () => {
|
|
const mockQuery = {
|
|
returning: jest.fn().mockResolvedValue([{ id: 123 }])
|
|
};
|
|
const result = await insertAndGetId(mockQuery);
|
|
expect(result).toBe(123);
|
|
});
|
|
|
|
test('should handle SQLite result format', async () => {
|
|
const mockQuery = {
|
|
returning: jest.fn().mockResolvedValue([456])
|
|
};
|
|
const result = await insertAndGetId(mockQuery);
|
|
expect(result).toBe(456);
|
|
});
|
|
});
|
|
}); |