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

application.yaml discouraged + NPE if no configuration #152

Open
ikucuze opened this issue Jul 24, 2024 · 1 comment · May be fixed by #153
Open

application.yaml discouraged + NPE if no configuration #152

ikucuze opened this issue Jul 24, 2024 · 1 comment · May be fixed by #153

Comments

@ikucuze
Copy link

ikucuze commented Jul 24, 2024

rafactor 7a8fbbe introduced an 'application.yaml' config file in springboot module.

Importing this dependency in a project having an application.yaml will mask the dependency one, so the configs in CamundaAutodeploymentProperties will be null.

This produces a NullPointer in CamundaProcessAutodeployment.deployCamundaResources(), due to auto unboxing of if (!properties.getEnabled()).

Hence, the behavior is different, depending on the project which import the dependency.

Defaults should be set in ConfigurationProperties file.

@ikucuze ikucuze changed the title aaplication.yaml discouraged + NPE if no configuration application.yaml discouraged + NPE if no configuration Jul 24, 2024
@ikucuze
Copy link
Author

ikucuze commented Jul 24, 2024

fix by PR #153

@ikucuze ikucuze linked a pull request Jul 24, 2024 that will close this issue
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 a pull request may close this issue.

1 participant