#24 updating some aspects to Bootstrap 5 and adding in both search lo…

[markidox]

…gs and download logs functionality

[JimKerslake]

Review from Claude:

The PR successfully implements the main features requested in issue #24:

1. CSV Export ✅ - Fully implemented with an Export action that downloads all logs as CSV
2. Search Functionality ✅ - Implemented with a search form that filters logs by URL and Message content
3. Better Log Analysis ✅ - Both features enable easier analysis without database access

🔴 Critical Issues

1. NoDb Implementation Incomplete
   - GetPagedSearchResults throws NotImplementedException in NoDb provider
   - GetExportData in NoDb appears incomplete (loads objects but doesn't add to result list)
   - This breaks functionality for users using file-based storage
2. CSV Export Security Issues
   - No CSV escaping/sanitization - vulnerable to CSV injection attacks
   - Missing quotes around fields that may contain commas, newlines, or special characters
   - Could corrupt CSV format if log messages contain commas
3. Search Implementation Flaws
   - Missing logLevel filter in EF Core search query (filter partially applied in count but not main query)
   - Uses ToString() on database fields unnecessarily
   - No case-insensitive search option
   - No wildcard/regex support as originally suggested
4. Performance Concerns
   - GetExportData loads ALL logs into memory at once - could cause OutOfMemory for large datasets
   - No streaming or pagination for export
   - No size limits or warnings about large exports

🟡 Minor Issues

1. Code Quality
   - Inconsistent error handling (swallows exceptions in Export)
   - Missing null checks in CSV generation
   - Bootstrap 5 migration mixed with feature implementation (should be separate commits)
2. Missing Features
   - No progress indicator for large exports
   - Search doesn't persist across pagination
   - No indication of search results count

Recommendations

1. Must Fix Before Merge:
   - Implement NoDb search functionality properly
   - Add CSV sanitization/escaping
   - Fix logLevel filter in search query
   - Consider streaming CSV export for large datasets
2. Should Consider:
   - Add export size limits or warnings
   - Implement case-insensitive search
   - Add search term highlighting in results
   - Separate Bootstrap upgrade from feature implementation

The PR partially addresses the requirements but has significant implementation issues that need fixing before
it's production-ready, particularly around the NoDb provider support and CSV security.