15 replies

Bug in effect-http's Swagger UI router with mountApp method

@TylorS discovered a bug in effect-http's swagger ui router - if an app with the router is mounted using

HttpServer.router.mountApp

HttpServer.router.mountApp

, the path prefix is not available in the

RouteContext

RouteContext

. The swagger router impl takes the prefix into consideration to construct correct path in the generated index.html. The

HttpServer.router.mount

HttpServer.router.mount

includes the prefix and I already tested with it everything works as expected. So there's a question whether this is a bug and

mountApp

mountApp

should be adjusted to add the prefix to the routes or whether this is intentional? @Tim Smart