3.7.2 broke "quarkus.minio.url" in devservices mode

[jendib]

Hi and thanks for this extension.
I'm relying on this code to get the current minio URL:

@ConfigProperty(name = "quarkus.minio.url")
String url;

but since 3.7.2 when Minio is started as a devservice (in unit tests) it's only filled by "localhost" instead of http://localhost:6878 (random port chosen by testcontainers).
I'm guessing it has to do with this recent change 204a8ae#diff-ac8ae1ef70059d67a0656e95d75eaeb3482ec3c7ec6a6284c1dd7b05bfe8badeR240

[jtama]

Oh I should have mention that change, could you please provider a sample of your usage ?

[jendib]

I just need to get the Minio URL, and because in a test context it's not a fixed value I don't know of any other way, is there?

[jtama]

Yes there is, you need to get the quarkus.minio.port injected as well. Please tell me of this works for you.

I will put up a PR to document this.

[jtama]

@jendib, does it work ?

[jtama]

Closing as I don't have more inputs. Please feel free to re-open if needed.

[jendib]

Hello sorry I'm waking up on this finally :)

I did the migration, however I still have one problem:
The guide says that quarkus.minio.url must start with http:// or https:// however the Dev Services section says that this configuration property will be

The container host name. It isn’t an URL, it doesn’t contains the exposed port. It’s value will certainly be localhost

There is a contradiction, how can the dev services mode set quarkus.minio.url to something not starting with http:// when it's a requirement?

[jtama]

The code is the truth. I forgot to update the doc. host is certainly a better name now.

But documentation aside, could you achieve what you wanted ?

[jendib]

So you are saying that I can put quarkus.minio.url=myminio.example.com and it would work even if the documentation says otherwise? If yes that would be great. Currently I have a code specific for dev services which is of course not great.

[jtama]

This is exactly what I say. I'll test that tomorrow, and amend the doc and examples.

[jtama]

Yup I have to amend the doc and mixed usage in the tests, but actually, the devservice uses localhost as value for the quarkus.minio.url property.

[jtama]

@jendib Can you confirm ?

[jtama]

@jendib Can you confirmed it works as expected with that clarification ?

[jendib]

@jtama It works perfectly, thanks a lot! It would be great indeed to rename quarkus.minio.url to quarkus.minio.host in the future.

[jtama]

I will, but I don't want to break other users... SO it will be a long process...