Fix web route prefix wrapped twice

[yeya24]

Signed-off-by: yeya24 yb532204897@gmail.com

• I added CHANGELOG entry for this change.
• Change is not relevant to the end user.

Fixes #2483

Changes

The web route prefix is registered twice.

Verification

Tested in my local env and here

[yeya24]

While I am trying to fix this, I found that web route prefix is not well handled for the routes other than /api/v1. I already implemented a fix for that, but I am not sure whether that fix should be included in this PR. So I opened #2486. Any ideas are welcomed!

[bwplotka]

Thanks on working on this ❤️

First of all do we know what commit introduced this bug? I think we should fix all, but on release-0.12 branch first so we can embed this fix to 0.12.1 or 0.12.2

cc @squat

[bwplotka]

Let us know when you wake up, we can talk offline.

[bwplotka]

Also question how @fengzixu or we can help with this if you are busy. (:

[squat]

Hmm, the original issue mentions #2208 but that doesn't seem like the cause to me

[squat]

@yeya24 let's make this PR against the release-0.12 branch and then we'll merge that branch back into master once we cut the release. Let me know if you want to make the PR, otherwise I can prepare it

[bwplotka]

I think Ben is US time zone, so if we want to have it sooner we need to do it alone.

…

On Tue, 21 Apr 2020 at 08:13, Lucas Servén Marín ***@***.***> wrote: @yeya24 <https://github.com/yeya24> let's make this PR against the release-0.12 branch and then we'll merge that branch back into master once we cut the release. Let me know if you want to make the PR, otherwise I can prepare it — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2484 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABVA3O4NAWO33ZGSWKUTDJDRNVBS7ANCNFSM4MMZS3XA> .

[yeya24]

@yeya24 let's make this PR against the release-0.12 branch and then we'll merge that branch back into master once we cut the release. Let me know if you want to make the PR, otherwise I can prepare it

@yeya24 let's make this PR against the release-0.12 branch and then we'll merge that branch back into master once we cut the release. Let me know if you want to make the PR, otherwise I can prepare it

Thanks. Please do it. ❤️

[squat]

Cool on it

[squat]

superseded by #2489
This fix will make in into master when the release-0.12 branch is merged into master after the release