FelixMarxIBM opened a new pull request, #458:
URL: https://github.com/apache/felix-dev/pull/458
Replace `getConfiguration()` call with `listConfigurations()` in
`ConfigInstaller.configurationEvent()` method to avoid unintentionally creating
new configurations. The getConfiguration() method creates a configuration if it
doesn't exist, which is not the desired behavior when simply checking for
existing configurations.
Changes:
- Use listConfigurations("(service.pid=" + pid + ")") instead of
getConfiguration(pid, "?")
- Add proper null and length checks for configuration arrays
- Update corresponding test mocks to reflect the corrected API usage
This fixes an issue with felix fileinstall reacting upon the CM_UPDATED
events and creating a configuration that was already deleted.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
