thruster-rs / Thruster

A fast, middleware based, web framework written in Rust
MIT License
1.07k stars 47 forks source link

Formalize route matching algorithm #75

Closed trezm closed 4 years ago

trezm commented 6 years ago

Given our tree structure, which I believe is fairly comprehensive, as it hasn't changed for a long time, I'd like to better formalize the algorithm for matching routes. Right now the matching code has been taped up many times and is smelling pretty bad. With a more formal algorithm we can drastically clean it up.

AlbanMinassian commented 6 years ago

ok, but not "urgent"

trezm commented 6 years ago

This is an issue that should be done before the 0.5 release. It's caused many headaches and we're at a point where the code has become complex enough that it's not immediately readable by me, let alone a new contributor to the project.

trezm commented 4 years ago

Clearly not a priority, closing until this becomes an issue again now that the algorithm has stabilized.