Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Unify the two configuration classes #20

Closed
mpreisler opened this issue Oct 17, 2015 · 2 comments
Closed

Unify the two configuration classes #20

mpreisler opened this issue Oct 17, 2015 · 2 comments
Milestone

Comments

@mpreisler
Copy link
Member

The image-scanner and openscap-daemon projects merged together and due to time constraints we have made some compromises.

One of them is the existence of applicationconfiguration next to config.Configuration. The code should be refactored to only use the latter. The hard work of removing the singleton has already been done by @baude in #13. The next step is passing self.system.config in the dbus_daemon class instead of letting the cve_scanner.Worker construct the applicationconfiguration instance.

@mpreisler mpreisler added this to the 0.2.0 milestone Aug 10, 2016
@mpreisler
Copy link
Member Author

#93 will get rid of this.

@matejak
Copy link
Contributor

matejak commented Oct 3, 2018

This is a sortof duplicate of #93

@matejak matejak closed this as completed Oct 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants