NET migartion #9

Merged

bpeskiewicz merged 23 commits from feature/cs into main 2026-04-21 20:37:20 +00:00

Conversation 0 Commits 23 Files changed 115 +3377 -7409

bpeskiewicz commented 2026-04-18 21:24:32 +00:00

Owner

Copy link

No description provided.

bpeskiewicz added 6 commits 2026-04-18 21:24:32 +00:00

chore: update .gitignore to include additional patterns for .NET, Node, Visual Studio, and OS generated files d5d5089d75

feat: migrate to .NET 9 Blazor architecture with integrated web UI ... c3085da9bf

Replaces the previous Node.js-based structure with a full .NET 9 Blazor
solution including Razor components, built-in authentication, and updated
project/tooling configuration. Removes legacy middleware and rate limiting
in favor of framework-integrated patterns.

chore: update CI, docs, and config for .NET 9 migration ... c0dd9fad06

Update CI pipelines, README, .env.example, and project settings to
reflect the migration from Node.js to ASP.NET Core 9 / Blazor.

refactor: adopt primary constructors, source-generated logging, and suppress CA1515/CA1031 analyzers ... 1b37e93aca

Modernize middleware, services, and test classes with C# primary
constructors and Microsoft.Extensions.Logging source generators for
high-performance logging. Suppress CA1515 for shared model types and
CA1031 for the global exception handler where catching all exceptions
is intentional.

refactor: remove service interfaces and consolidate analyzer config ... 702dda0324

Remove IAuthService, IFileService, and IDownloadTokenService interfaces
in favor of direct concrete type injection. Add InternalControllerProvider
to support internal controller discovery. Move analyzer settings to
Directory.Build.props and add .editorconfig files for consistent formatting.

refactor: apply analyzer fixes to AppException and test files ... 7dd665f8a5

Adopt primary constructor for AppException with CA1032 suppression,
add ConfigureAwait(true) to async test calls, seal RateLimitingTests,
and register tests/.editorconfig in solution.

bpeskiewicz added 2 commits 2026-04-19 19:44:04 +00:00

test: add E2E test project with auth, download, and file table tests c54e51be74

refactor: extract configuration from Program.cs to AppConfiguration class ... ec3bc772b1

- Move configuration loading and database setup from Program.cs
- Create AppConfiguration class to centralize configuration logic
- Simplify Program.cs by delegating configuration management
- Maintain E2E test factory changes

bpeskiewicz added 1 commit 2026-04-19 20:06:30 +00:00

fix: resolve exitcode 127 in Docker test workflow ... 6879f1133b

Replace pwsh with dotnet exec for Playwright browser installation
to ensure compatibility with Docker containers

bpeskiewicz added 1 commit 2026-04-19 20:12:53 +00:00

refactor: fix Playwright.NET installation in test workflow ... d5307d5753

- Update to use official Playwright CLI instead of .NET SDK equivalent
- Install Playwright browsers using playwright install command
- Add Playwright.NET tool installation step before browser installation

bpeskiewicz added 1 commit 2026-04-19 20:21:32 +00:00

fix: use full path for playwright CLI in test workflow 0fcc834b25

bpeskiewicz added 1 commit 2026-04-19 20:25:09 +00:00

fix: specify project path for playwright install in test workflow 12b954041a

bpeskiewicz added 1 commit 2026-04-19 20:28:31 +00:00

fix: use full .csproj path for playwright install 5effdb816b

bpeskiewicz added 1 commit 2026-04-19 20:32:56 +00:00

fix: use build output playwright.sh instead of deprecated global CLI 60646da8e0

bpeskiewicz added 1 commit 2026-04-19 20:37:01 +00:00

fix: use pwsh to run playwright.ps1 from build output f783618eaa

bpeskiewicz added 1 commit 2026-04-19 20:40:04 +00:00

fix: install PowerShell before running playwright.ps1 54ba75ccbd

bpeskiewicz added 1 commit 2026-04-19 20:42:44 +00:00

fix: install PowerShell via dotnet tool for ARM64 compatibility 7750c1fa38

bpeskiewicz added 1 commit 2026-04-19 20:45:43 +00:00

fix: use npx to install Playwright browsers directly ... d4f3009ac7

Skips the broken PowerShell/dotnet tool approaches. Playwright browsers
are shared between Node.js and .NET packages, so npx installs the same
browsers the .NET package needs.

bpeskiewicz added 1 commit 2026-04-19 21:01:06 +00:00

fix: removed windows hardcoded root path ff36c4ae35

bpeskiewicz added 4 commits 2026-04-21 16:15:40 +00:00

feat: add logging and pagination support for file operations ... 7736467411

- Add structured logging for login events with remote IP tracking
- Implement pagination for file listing endpoints with configurable limits
- Add logging for file downloads and link generation
- Configure session duration via AuthOptions instead of hardcoded value
- Add input validation for pagination parameters

refactor: optimize download token service and files controller ... f5ed000184

- Improved download token service with better error handling and logging
- Updated files controller to use async stream reading for better performance
- Added null checks and improved exception handling

refactor: convert DTOs to immutable records ... 6ee3673188

Convert all DTO classes to records with inline constructor parameters for better immutability guarantees and reduced boilerplate. Updated all instantiations to use constructor syntax.

- ApiErrorResponse: record with Error and Message properties
- NavBreadcrumbItem: record with Title, Path, and Disabled properties
- FileItem: record with Name, Path, Type, Size, and Modified properties
- HealthResponse: record with Status and Uptime properties
- LoginRequest: record with Username and Password properties

All E2E tests pass (11/11).

feat: implement Result/Monad pattern for error handling ... eb621169f7

Add Result<T> type and related infrastructure to provide consistent error handling across controllers and services. Includes Result filter for MVC integration and updates to auth flows.

bpeskiewicz merged commit 279e5124fb into main 2026-04-21 20:37:20 +00:00

bpeskiewicz referenced this pull request from a commit 2026-04-21 20:37:21 +00:00

Merge pull request 'NET migartion' (#9) from feature/cs into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

bpeskiewicz/file-gateway!9

No description provided.