Skip to content
This repository has been archived by the owner on Jan 24, 2021. It is now read-only.

Fixes for strange routing behaviour with optionals. #2683

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixes for strange routing behaviour with optionals. #2683

wants to merge 1 commit into from

Commits on Jan 18, 2017

  1. Fixes for strange routing behaviour with optionals. 

     The routing engine incorrectly routes to longer urls in preference
 of the score, plus it falls back on routing to the last added matching routes
 instead of the first.

 I've fixed the sorting comparison method and added some test coverage to
 show the particular use cases where I found issues.
    @PhilipSkinner
    PhilipSkinner authored and Philip Skinner committed Jan 18, 2017
    Configuration menu
    Copy the full SHA
    1700665 View commit details
    Browse the repository at this point in the history