This PR changes the server property of the OpenAPI spec that we serve to make it work when you provide a proxy base path.
The way we have the spec set up, we'll only ever have a single server, so there's no point in providing the base url the way we used to. According to the OpenAPI spec, an empty list will make it fall back to the default, which is /.
The only difference you'll see in the UI from before is that the servers dropdown is gone.
Discussion
Ideally, I think we would like the /base/path to be part of the server base path and not a prefix to each and every API endpoint. However, as we have seen in Unleash (https://github.com/Unleash/unleash/pull/2079), that isn't trivial.
This PR should fix the problem for now. If we want to, we can make the improvement later.
This PR changes the
serverproperty of the OpenAPI spec that we serve to make it work when you provide a proxy base path.The way we have the spec set up, we'll only ever have a single server, so there's no point in providing the base url the way we used to. According to the OpenAPI spec, an empty list will make it fall back to the default, which is
/.The only difference you'll see in the UI from before is that the servers dropdown is gone.
Discussion
Ideally, I think we would like the
/base/pathto be part of the server base path and not a prefix to each and every API endpoint. However, as we have seen in Unleash (https://github.com/Unleash/unleash/pull/2079), that isn't trivial.This PR should fix the problem for now. If we want to, we can make the improvement later.
closes #125