Skip to content

Commit

Permalink
BUG: ctl: fix default logging path
Browse files Browse the repository at this point in the history
the default logging path is not opt, it's the constant from the top
__init__.py, which is different for deb/rpm packages

fixes #2091
  • Loading branch information
Sebastian Wagner committed Sep 1, 2021
1 parent cbf5335 commit eba287d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ CHANGELOG
- `intelmqctl`:
- Also honour parameters from environment variables (PR#2068 by Sebastian Wagner, fixes #2063).
- Fix management actions (start/stop/status/reload/restart) for groups (PR#2086 by Sebastian Wagner, fixes #2085).
- Do not use hardcoded logging path in `/opt/intelmq`, use the internal default instead (PR#2092 by Sebastian Wagner, fixes #2091).

### Contrib

Expand Down
2 changes: 1 addition & 1 deletion intelmq/bin/intelmqctl.py
Original file line number Diff line number Diff line change
Expand Up @@ -704,7 +704,7 @@ def __init__(self, interactive: bool = False, return_type: str = "python", quiet

# set default values for the logging handler
self.parameters.logging_handler = 'file'
self.parameters.logging_path = '/opt/intelmq/var/log'
self.parameters.logging_path = DEFAULT_LOGGING_PATH

# Try to get logging_level from defaults configuration, else use default (defined above)
defaults_loading_exc = None
Expand Down

0 comments on commit eba287d

Please sign in to comment.