The configuration quarkus.smallrye-openapi.servers ignores the order in the swagger ui

[wlybe]

Describe the bug

Setting servers on the application properties is randomly ordered.

These two settings show the servers in the same order on the swagger ui:

%dev:
  quar:kus:
    smallrye-openapi:
      servers: https://my.produciton.org, http://localhost:8080

or

%dev:
  quar:kus:
    smallrye-openapi:
      servers: http://localhost:8080, https://my.produciton.org

Maybe this is because on application.properties or application.yaml: quarkus.smallrye-openapi.servers is types java.util.Set. on openapi.yaml, server: array (and not a set)

Expected behavior

https://my.produciton.org appears first on the list in SwaggerUI with this configuration

%dev:
  quar:kus:
    smallrye-openapi:
      servers: https://my.produciton.org, http://localhost:8080

http://localhost:8080 appears first on the list in SwaggerUI with this configuration

%dev:
  quar:kus:
    smallrye-openapi:
      servers: http://localhost:8080, https://my.produciton.org

Actual behavior

These two settings show the servers in the same order on the swagger ui:

%dev:
  quar:kus:
    smallrye-openapi:
      servers: https://my.produciton.org, http://localhost:8080

or

%dev:
  quar:kus:
    smallrye-openapi:
      servers: http://localhost:8080, https://my.produciton.org

How to Reproduce?

1. add this configuration to application.yaml

%dev:
  quar:kus:
    smallrye-openapi:
      servers: https://my.produciton.org, http://localhost:8080

2. open the swagger UI
3. Do the same by changing the servers

%dev:
  quar:kus:
    smallrye-openapi:
      servers: https://my.produciton.org, http://localhost:8080

4. On swaggerUI, the servers order do not change

Output of uname -a or ver

No response

Output of java -version

openjdk 11.0.17 2022-10-18 LTS

GraalVM version (if different from Java)

No response

Quarkus version or git rev

2.14.1.Final

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.8.6

Additional information

No response

[quarkus-bot]

/cc @EricWittmann, @Ladicek, @MikeEdgar, @jmartisk, @phillip-kruger, @radcortez

[gsmet]

The Set in the config is not really used but I will change it once we have this fixed in SmallRye OpenAPI, where I think the problem lies. I created a pull request here: smallrye/smallrye-open-api#1323 .

[MikeEdgar]

This one was resolved in SR OpenAPI 3.1.1 with #29815

[MikeEdgar]

@gsmet bump - this can be closed as complete.

[wlybe]

Indeed, I tested on Quarkus 2.16.4.Final and it looks ok :)

Thanks !