Skip to content

Fix: correct SCCM config encryption and query handling - #186

Merged
RomainLvr merged 2 commits into
mainfrom
fix_config
Jul 28, 2026
Merged

Fix: correct SCCM config encryption and query handling#186
RomainLvr merged 2 commits into
mainfrom
fix_config

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes #N/A
  • Some SCCM configuration fields were stored and displayed in plain text instead of being encrypted like the other credential fields, and the default certificate validation setting was not enabled by default.
  • Device identifiers used when building database queries, XML files and file paths were not validated or parameterized, and connection error messages were not escaped before being displayed.
  • This fix aligns these fields and code paths with the existing patterns already used elsewhere in the plugin.

Screenshots (if appropriate):

@Rom1-B
Rom1-B requested a review from RomainLvr July 28, 2026 13:08
@RomainLvr
RomainLvr merged commit 0ae9f7a into main Jul 28, 2026
3 checks passed
@RomainLvr
RomainLvr deleted the fix_config branch July 28, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants