Quarkus: -Dquarkus.http.host=0.0.0.0 force set in s2i and jib despite it being the default

[maxandersen]

Describe the bug
just noticed that -Dquarkus.http.host=0.0.0.0 being set on various jvm/native arguments on https://quarkus.io/guides/all-config for s2i and jib.

Since https://github.com/quarkusio/quarkus/pull/1307 that been the default value anyway and it also means any setting in application.properties will get ignored (afaics).

Is that really the intention ?

I would have expected default values to be used or what set in application.properties. I wonder if it was just done this way as the default was not set to make it work out of box back then....consequence now is though if we change it we do run the risk of breaking users since i their quarkus.http.host is set in application.properties - it would have been ignored by now for s2i and jib usecases.

We should though fix this imo (assuming there are no good reason we force them) and document it clearly.

[quarkusbot]

/cc @geoand

[geoand]

I also found it weird as well... We should definitely get rid of it as it's the default which is very unlikely to change at this point

[maxandersen]

well even it was to change; isn't ones expectation that settings in application.properties should be listened too ?

[geoand]

Yes for sure and this setting does indeed violate that expectation (although in all this time no one I have seen has tried to set quarkus.http.host and run into this - probably because no one is setting quarkus.http.host :)).

[maxandersen]

yeah, very likely and thus just something to clean up to not trigger my ocd for clean settings ;)

[geoand]

I'll do this over the weekend if you don't beat me to it :)

[geoand]

https://github.com/quarkusio/quarkus/issues/11918 takes care of this, but I am thinking we probably want to fix it for docker as well