Vertx-web: colon in Path generate "resource not Found"

[micjaesc]

Version

• vert.x core:3.5.1
• vert.x web:3.5.1

Context

We have to expose API with path, which includes : (colon) like : domain/v1/user/aa:bb/subscriptions

It looks our router is not able to match to any of our endpoint
Fragment from openapi.yaml (yes, we are using OpenAPI3RouterFactory)

paths:
  /v1/user/{uuid}/subscriptions:

PS
Present response : "Resource not found"

[slinkydeveloper]

The character : should be encoded https://perishablepress.com/stop-using-unsafe-characters-in-urls/

[micjaesc]

forget to mention, of course it was encoded, just pasted here decoded
Anyway testing via domain/v1/user/a%3Ab/subscriptions and It gives same result (Resource not found)

[slinkydeveloper]

i will look at it

[micjaesc]

some hint, when using regular vertx router , then all is fine

val router = routerFactory.router
            with(router) {
                get("/v1/user/:UUID/subscriptions")

only when switching to OpenAPI3RouterFactory, problem appears

Additional info. yml defintion says about this parameter as

      in: path
      required: true
      schema:
        type: string

[slinkydeveloper]

Please check this PR: https://github.com/vert-x3/vertx-web/pull/872

[micjaesc]

thank you. Looks promising.

[micjaesc]

I can see it will be released with 3.6.0 , right?
So, what is the schedule for it, estimation of deploy/release date?

[micjaesc]

is there chance for inter release like 3.5.2 ?

[micjaesc]

after fix, I will be able to use : or only encoded %3A or both ?

[slinkydeveloper]

I don't know when the next version will be release. I think that : won't work because it's a reserved character that should be escaped

[slinkydeveloper]

Just watch this regex https://github.com/vert-x3/vertx-web/pull/872/files#diff-ef77bbac586ab216b4fc91092e1085fcR91

[micjaesc]

In the spec (https://tools.ietf.org/html/rfc3986#section-3.3) we found out that ':' is technically valid in the _path_ segment of a url - so we do not need encode

[slinkydeveloper]

Please look at field allowReserved (that we don't support, we assert that is always false) https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.1.md#fixed-fields-10

[slinkydeveloper]

I've created a new PR that refactor the path generation: https://github.com/vert-x3/vertx-web/pull/884