Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Config property expects to have hyphen before digit #38582

Closed
mannukalra opened this issue Feb 5, 2024 · 4 comments · Fixed by #38610
Closed

Config property expects to have hyphen before digit #38582

mannukalra opened this issue Feb 5, 2024 · 4 comments · Fixed by #38610
Assignees
Labels
area/config kind/bug Something isn't working
Milestone

Comments

@mannukalra
Copy link

mannukalra commented Feb 5, 2024

Describe the bug

While trying to update Quarkus from 3.2.9.Final to 3.6 onwards have come across following issue related to config-
Config property expects to have hyphen before digit,
Example in config java file-

public interface ParenttConfig {
  interface Id {
          String id();
  }
  interface XChannels {
          Id acrossB2b();
  }
  ..........
}

config yaml-

parentt:
    x-channels:
        across-b2b:
            val: xxxxxx

throws an exception on start-up

java.util.NoSuchElementException: SRCFG00014: The config property parentt.x-channels.across-b-2b.val is required but it could not be found in any config source

Expected behavior

'acrossB2b' in java should be accepted as 'across-b2b' from yaml.

Actual behavior

'acrossB2b' in java is expecting to have 'across-b-2b' from yaml.

How to Reproduce?

Custom Yaml config file with a number in the key name, occurs on application start-up.

Output of uname -a or ver

No response

Output of java -version

17.0.7

Quarkus version or git rev

3.6.0

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

mvn 3.8.4

Additional information

No response

@mannukalra mannukalra added the kind/bug Something isn't working label Feb 5, 2024
@geoand
Copy link
Contributor

geoand commented Feb 5, 2024

cc @radcortez

@radcortez radcortez self-assigned this Feb 6, 2024
@quarkus-bot quarkus-bot bot added this to the 3.9 - main milestone Feb 7, 2024
@gsmet gsmet modified the milestones: 3.9 - main, 3.7.2 Feb 7, 2024
@mannukalra
Copy link
Author

Thanks for the prompt actions, if possible please help with the expected release date for 3.7.2?

@radcortez
Copy link
Member

We may still be a few days away.

To move forward, you can manually update the SmallRye Config version, which fixes the issue:

<dependency>
  <groupId>io.smallrye.config</groupId>
  <artifactId>smallrye-config-common</artifactId>
  <version>3.5.4</version>
</dependency>
<dependency>
  <groupId>io.smallrye.config</groupId>
  <artifactId>smallrye-config-core</artifactId>
  <version>3.5.4</version>
</dependency>

@gsmet
Copy link
Member

gsmet commented Feb 8, 2024

3.7.2 will be released today if the release gods are with us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/config kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants