fix #146

[maikebing]

No description provided.

[Copilot]

Pull Request Overview

This PR addresses issue #146 by ensuring that route URL concatenations correctly handle trailing slashes in the virtual path root. The changes update authentication paths, controller routes, and file server request paths to conditionally insert a slash if needed.

• Updated login and access denied paths in ServiceCollectionExtensions.cs.
• Modified controller route paths and file server request path in ApplicationBuilderExtensions.cs.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/SilkierQuartz/Configuration/ServiceCollectionExtensions.cs	Adjusts authentication route paths for correct URL formatting.
src/SilkierQuartz/Configuration/ApplicationBuilderExtensions.cs	Updates controller routes and file server path for consistent URL construction.

Comments suppressed due to low confidence (2)

src/SilkierQuartz/Configuration/ServiceCollectionExtensions.cs:44

• [nitpick] The logic for conditionally appending a slash is repeated; consider extracting it into a helper method to improve maintainability and reduce duplication.

cfg.LoginPath = "${options.VirtualPathRoot}{(options.VirtualPathRoot.EndsWith('/') ? "" : "/")}Authenticate/Login";

src/SilkierQuartz/Configuration/ApplicationBuilderExtensions.cs:91

• [nitpick] The conditional logic for ensuring the correct separator is repeated; extracting this logic into a shared utility could enhance code clarity and maintainability.

endpoints.MapControllerRoute(nameof(SilkierQuartz), "${options.VirtualPathRoot}{(options.VirtualPathRoot.EndsWith('/')?"":"/")}{controller=Scheduler}/{action=Index}");