yahoo/fluxible

[fluxible-router] case-sensitive routing

Open

#435 geöffnet am 27. Apr. 2016

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.816 Stars) (233 Forks)batch import
enhancementhelp wanted

Beschreibung

the routing in fluxible-router is case insensitive currently. i'd like to use case-sensitive routing..

it seems that fluxible-router uses routr, which in turn uses path-to-regexp, which has an option for this, but it's not possible to pass down this option to it.

basically this line https://github.com/yahoo/routr/blob/master/lib/router.js#L38 needs to get an additional {sensitive:true} param at the end.

what would be the best way to achieve this? (to be able to choose case-sensitive routing)

Contributor Guide

Tech Stack
javascriptnodejs
Domain
full stack
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic JavaScriptNode.jsRouting concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
Investigate routr's router.js at line 38 to add the `{sensitive:true}` option. Check if routr has a way to pass options from fluxible router. Consult the path to regexp documentation for the sensitive flag. Consider adding a configuration option to fluxible router to enable case sensitive routing.