The pathParse function in RescriptReactRouter.res currently treats single slash and double slash URLs identically, which causes routing conflicts and prevents proper URL differentiation.

Offen Anfängerfreundlich

#145 erstellt am 17.9.2025 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 65/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: react
Bereich: frontend

Rechercherichtung

Beginne in RescriptReactRouter.res bei der Funktion pathParse und untersuche die split- und filter-Logik für durch Schrägstriche getrennte Pfade. Überprüfe das Verhalten für /analytics-transaction und //analytics-transaction und bestätige anschließend, dass die resultierenden Listen das führende leere Segment nur für die URL mit doppeltem Schrägstrich beibehalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Current Behavior

// Both URLs return the same parsed result
pathParse("/analytics-transaction") → list{"analytics-transaction"}
pathParse("//analytics-transaction") → list{"analytics-transaction"} // Problem!

Expected Behavior

// URLs should parse differently to enable proper routing
pathParse("/analytics-transaction") → list{"analytics-transaction"}
pathParse("//analytics-transaction") → list{"", "analytics-transaction"} // Should preserve leading empty string

Impact

Cannot differentiate between /path and //path in routing logic
Causes "Page Not Found" errors for valid double slash URLs
Breaks routing patterns that rely on double slash prefixes

Proposed Solution

Modify the filter logic to preserve the first empty string while removing others:
// Current problematic code
raw->Js.String2.split("/")->Js.Array2.filter(item => item->Js.String2.length != 0)->arrayToList

// Proposed fix
let splitArray = raw->Js.String2.split("/")
let filteredArray = []
splitArray->Js.Array2.forEachi((item, index) => {
if item->Js.String2.length != 0 || index == 0 {
filteredArray->Js.Array2.push(item)->ignore
}
})
filteredArray->arrayToList

Vorherrschende Sprache: ReScript
Sterne: 517
Forks: 45
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus rescript-lang/rescript-react

Allow arbitrary data-* attributes on DOM elements in JSX Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100

rescript-lang/rescript-react#152 · 3 Reaktionen · Erstellt am 07.3.2026
[RFC] Support ref as prop for React 19 compatibility Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100

rescript-lang/rescript-react#147 · 2 Kommentare · Erstellt am 28.12.2025
ReScript-aware `memo` Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

rescript-lang/rescript-react#111 · 3 Kommentare · Erstellt am 21.4.2024
Implicit type conversion? Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

rescript-lang/rescript-react#105 · Erstellt am 30.11.2023
Scroll Restoration Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

rescript-lang/rescript-react#104 · 1 Reaktion · Erstellt am 20.11.2023

Alle Issues in rescript-lang/rescript-react