-
Notifications
You must be signed in to change notification settings - Fork 73
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
Update logmanager.sh #9702
base: main
Are you sure you want to change the base?
Update logmanager.sh #9702
Conversation
Update for Janssen arch. Signed-off-by: mzico <[email protected]>
DryRun Security SummaryThe pull request modifies a log management script, "logmanager.sh", to work with the Janssen application instead of the previous Gluu application, and expands the list of directories where log files are managed, while ensuring proper access controls and permissions are in place to prevent unauthorized access or modification of log files. Expand for full summarySummary: The changes in this pull request appear to be a modification to a log management script called "logmanager.sh". The script is responsible for compressing and deleting log files that are older than a certain number of days. The key changes include updating the script to work with the Janssen application instead of the previous Gluu application, as well as expanding the list of directories where log files are managed. From an application security perspective, the changes in this script are not particularly concerning, as log management is a common and necessary task for any application. However, it's important to ensure that the script is only deleting the intended files and that proper access controls and permissions are in place to prevent unauthorized access or modification of log files. Files Changed:
Code AnalysisWe ran Riskiness🟢 Risk threshold not exceeded. |
Quality Gate passed for 'jans-linux-setup'Issues Measures |
Update for Janssen arch.
Prepare
Description
Target issue
closes #issue-number-here
Implementation Details
Test and Document the changes
Please check the below before submitting your PR. The PR will not be merged if there are no commits that start with
docs:
to indicate documentation changes or if the below checklist is not selected.Closes #9703,