Skip to content

fix(centreontrapd): create sqlite database file with good rights#3011

Open
s-duret wants to merge 3 commits intodevelopfrom
MON-192523
Open

fix(centreontrapd): create sqlite database file with good rights#3011
s-duret wants to merge 3 commits intodevelopfrom
MON-192523

Conversation

@s-duret
Copy link
Copy Markdown
Contributor

@s-duret s-duret commented Dec 16, 2025

Description

Create the centreontrapd.sdb file if it doesn't exist and add the good rights.

Fixes # MON-192523

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 23.10.x
  • 24.04.x
  • 24.10.x
  • 25.10.x
  • master

How this pull request can be tested ?

Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.

Any relevant details of the configuration to perform the test should be added.

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

🐛 Bugfixes

  • Created centreontrapd.sdb file when missing and set permissions
  • Corrected conditional check to use intended variable in trapd

More info

@s-duret s-duret requested a review from a team as a code owner December 16, 2025 13:13
@s-duret s-duret requested a review from sdepassio December 16, 2025 13:13
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 16, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • coderabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b9970f4a-4da7-4612-a461-4d9d7222a7c6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch MON-192523

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Detailsd6c2aca3-1789-46bc-a222-9af7cfca067c

Great job! No new security vulnerabilities introduced in this pull request

@Tpo76
Copy link
Copy Markdown
Contributor

Tpo76 commented Apr 15, 2026

🤖 This is an automatic comment generated by an internal tool.

Complexity Classification: Standard

Justification: Automatic classification failed; defaulting to standard.

Metric Value
WIP Points 2
SLA Track Standard SLA (2 work days)
First review SLA 1 work day

A maintainer can override this classification at any time.

Comment thread perl-libs/lib/centreon/script/centreontrapd.pm Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants