Clean up usage of hard coded hosts that use containers

[jamesnetherton]

No description provided.

[zhfeng]

It also needs to do in PahoMqtt5TestResource.java

camel-quarkus/integration-tests/paho-mqtt5/src/test/java/org/apache/camel/quarkus/component/paho/mqtt5/it/PahoMqtt5TestResource.java

Lines 80 to 81 in 5f4d5b3

	result = CollectionHelper.mapOf(
	"paho5.broker.tcp.url", "tcp://localhost:" + port);

and

camel-quarkus/integration-tests/paho-mqtt5/src/test/java/org/apache/camel/quarkus/component/paho/mqtt5/it/PahoMqtt5TestResource.java

Lines 102 to 109 in 5f4d5b3

	if (!useFixedPort) {
	result = CollectionHelper.mapOf(
	"camel.component.paho-mqtt5.username", MQTT_USERNAME,
	"camel.component.paho-mqtt5.password", MQTT_PASSWORD,
	"paho5.broker.tcp.url", String.format("tcp://localhost:%d", container.getMappedPort(TCP_PORT)),
	"paho5.broker.ssl.url", String.format("ssl://localhost:%d", container.getMappedPort(SSL_PORT)),
	"paho5.broker.ws.url", String.format("ws://localhost:%d", container.getMappedPort(WS_PORT)));
	}

[jamesnetherton]

It also needs to do in PahoMqtt5TestResource.java

Thanks, I made work like the other paho test. The downside is that when using a custom HostnameVerifier org.eclipse.paho.mqttv5.client.internal.NetworkModuleService does reflection on an internal JDK class java.net.URI. So I had to --add-opens to deal with that.

[zhfeng]

Yeah, that makes sense.

[zhfeng]

Do we need to add some notes in docs of paho-mqtt5? is it only needed in ssl cases?

[jamesnetherton]

I went back to look at it again. And the test is actually passing without it.

Maybe I previously had something misconfigured that triggered the problem code in NetworkModuleService.

I can revert the --add-opens and create follow up issue to investigate further as there's potentially an issue there for some scenarios (to be verified). WDYT?

[zhfeng]

Sure, it looks good to me.