Added wildcard feature

[dharshan-0]

Related issue or discussion

Description

It closes #545 by adding wildcard routing feature.

It uses this {*param_name} to define wildcard.

Route::with_handler_and_name("/random/{*path}", generate_random, "generate-random"),

Type of change

• Bug fix
• New feature
• Documentation
• Refactor / cleanup
• Performance improvement
• Other (describe above)

Checklist

• I've read the contributing guide
• Tests pass locally (just test-all)
• Code passes clippy (just clippy)
• Code is properly formatted (cargo fmt)
• New tests added (regression test for bugs, coverage for new features)
• Documentation (both code and site) updated (if applicable)

[codecov]

Codecov Report

❌ Patch coverage is 98.18182% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
cot/src/router/path.rs	98.18%	0 Missing and 1 partial ⚠️

Flag	Coverage Δ
rust	90.26% <98.18%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
cot/src/router/path.rs	99.39% <98.18%> (-0.26%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ElijahAhianyo]

@dharshan-0 Thanks a lot for your contribution. This is a great start! Let's address the comments, and it's good to merge.

[dharshan-0]

@ElijahAhianyo Thanks for your comments, I will look into it.

[ElijahAhianyo]

@dharshan-0 Thanks for your contribution once again!

[seqre]

Overall, it's a great job, thank you for your contribution!

I see one possible clash with overlapping routes, eg. /foo/{*path} and /foo/bar. To be honest, I'm not sure which route the framework would use atm. I think that it should first use the qualified path if it exists, and if it doesn't, then match the wildcard one.

I'd like for you to add a test(s) checking that functionality (also with different ordering of route definition in Router::with_urls) and if it does not work like that, to adapt the code. Also, mention that in the documentation, please.

Once that's done, I'll be happy to approve!

[dharshan-0]

Thats a valid point, I will look into that