dotnet / dotnet/source-indexer

Since we're covering `pathmap` now, we can end up with more interesting scenarios where it "conflicts" with `serverpath`.

Open
#303 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
55
Forks
46
Avg merge
23h 42m
Merged PRs (30d)
15

Description

Since we're covering `pathmap` now, we can end up with more interesting scenarios where it "conflicts" with `serverpath`.

We already had an issue here where `serverPath` was effectively "first one wins" rather than "longest prefix wins" and this extends that if `pathMap` includes a path that's a subfolder of some user specified `serverPath` (making it less obvious)

Since it's pre-existing, we can probably handle it in a follow up PR; but I figured it's worth calling out.

---

There might also be some normalization considerations, particularly with the keys ending with `\` but URLs not strictly ending with `/` and with Roslyn's robust handling/normalization of paths vs the more naive handling htmlgenerator has.

_Originally posted by @tannergooding in https://github.com/dotnet/source-indexer/pull/296#discussion_r3863391249_

Contributor guide

No contributing guide indexed for this repository

Research direction

No source file or test is named. Start by tracing how pathmap and serverpath are handled, then compare the existing behavior with Roslyn path normalization and htmlgenerator URL handling. Done means the conflict and path-normalization expectations are explicitly determined and verified.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.