Skip to content

Comments

Some compatibility fixes for current releases#341

Open
gsmet wants to merge 2 commits intomainfrom
compatibility-with-main
Open

Some compatibility fixes for current releases#341
gsmet wants to merge 2 commits intomainfrom
compatibility-with-main

Conversation

@gsmet
Copy link
Member

@gsmet gsmet commented Feb 17, 2026

Here are some compatibility fixes with current releases, in the hope it would make Ecosystem CI green again.

Unfortunately, with these changes, the tests can't connect to the broker anymore. A closer look from someone familiar with the extension might be needed, but I thought I might as well push that as a first step in the right direction.

@gsmet gsmet requested a review from a team as a code owner February 17, 2026 09:51
@bpasson
Copy link
Contributor

bpasson commented Feb 17, 2026

@gsmet I'll have a look if I can make it all work.

@bpasson
Copy link
Contributor

bpasson commented Feb 19, 2026

@gsmet I'm having a weird issue getting it to work again. Something goes wrong with the configuration properties. The property quarkus.rabbitmqclient.tls.enabled is mapped to RabbitMQClientConfig.TlsConfig.enabled() annotated with @WithDefault("false"). This works fine in 3.30.x, but breaks in 3.31.x. In 3.31.x the value is assigned true with source BuildTime RunTime Fixed in stead of expected false from the source DefaultValuesConfigSource. Do you have any idea what I am doing wrong here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants