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

The Wazuh indexer ISM log file should not be located in the tmp directory #2657

Closed
4 tasks done
rauldpm opened this issue Nov 29, 2023 · 2 comments · Fixed by #2732
Closed
4 tasks done

The Wazuh indexer ISM log file should not be located in the tmp directory #2657

rauldpm opened this issue Nov 29, 2023 · 2 comments · Fixed by #2732
Assignees
Labels
level/task Subtask issue type/bug Bug issue

Comments

@rauldpm
Copy link
Member

rauldpm commented Nov 29, 2023

Description

It has been detected that the Wazuh indexer package is creating a log file in /tmp/wazuh-indexer, this is not correct and it should use the package's log directory: /var/log/wazuh-indexer

LOG_FILE="/tmp/wazuh-indexer/ism-init.log"

A log file should be permanent since a reboot of the machine would lose this data.

Additionally, on certain systems, the tmp directory has the noexec flag, which prevents the installation of the package, this has been reported by many users, and an issue was opened to change it: #1539

image

Related PR: https://github.com/wazuh/wazuh-packages/pull/2606/files

Tasks

  • Change the ISM log file to /var/log/wazuh-indexer

Validation

  • The operation of the package is correct
  • Logs are created and maintained correctly
  • Temporary directory is not used
@rauldpm rauldpm added level/task Subtask issue type/bug Bug issue labels Nov 29, 2023
@AlexRuiz7 AlexRuiz7 linked a pull request Dec 18, 2023 that will close this issue
30 tasks
@AlexRuiz7 AlexRuiz7 removed a link to a pull request Dec 18, 2023
30 tasks
@Deblintrake09 Deblintrake09 self-assigned this Jan 3, 2024
@Deblintrake09 Deblintrake09 linked a pull request Jan 3, 2024 that will close this issue
3 tasks
@pro-akim
Copy link
Member

pro-akim commented Jan 3, 2024

Review Notes

LGTM

@davidjiglesias
Copy link
Member

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level/task Subtask issue type/bug Bug issue
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants