Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changed tokenizer to match std route match behavior; do not trimright… issue #497 #511

Merged
merged 1 commit into from Oct 19, 2022

Conversation

emicklei
Copy link
Owner

… the path

@codecov-commenter
Copy link

codecov-commenter commented Sep 30, 2022

Codecov Report

Merging #511 (50c3bf5) into v3 (67c9f7e) will decrease coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##               v3     #511      +/-   ##
==========================================
- Coverage   69.80%   69.61%   -0.19%     
==========================================
  Files          28       28              
  Lines        1606     1606              
==========================================
- Hits         1121     1118       -3     
- Misses        413      414       +1     
- Partials       72       74       +2     
Impacted Files Coverage Δ
route.go 85.07% <100.00%> (ø)
path_processor.go 91.42% <0.00%> (-8.58%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@starzhanganz
Copy link

@emicklei is that possible to get this merged? thanks.

@emicklei emicklei merged commit ec34605 into v3 Oct 19, 2022
brandond added a commit to brandond/k3s that referenced this pull request May 23, 2023
Fix regression in legacy API prefix, until upstream pulls in support for MergePathStrategy from emicklei/go-restful#511

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants