Add Adobe plugins to software inventory - #50039
Conversation
|
@coderabbitai full review |
✅ Action performedFull review finished. |
WalkthroughAdds the Possibly related issues
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@cmd/osquery-perf/agent.go`:
- Around line 3919-3931: Add startup validation for
commonAdobePluginsSoftwareCount and uniqueAdobePluginsSoftwareCount before
agents are created, alongside the existing software uninstall validation. Reject
negative values before softwareAdobePlugins reaches make, using the same
validation and failure behavior already used for related software counts.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: f5b67c49-32d4-4604-88b5-10bd3b862e63
📒 Files selected for processing (5)
cmd/osquery-perf/agent.goserver/service/osquery.goserver/service/osquery_test.goserver/service/osquery_utils/queries.goserver/service/osquery_utils/queries_test.go
Resolves #49060 Ingest fleetd's adobe_plugins table as a new `adobe_plugins` software source, so Adobe Creative Cloud CEP and UXP extensions show up in inventory on macOS and Windows hosts. osquery-perf gets -common_adobe_plugins_software_count and -software_adobe_plugins_query_fail_prob.
073644d to
15e8e24
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 45414-software-inventory-vulnerabilities-add-adobe-plugins #50039 +/- ##
=============================================================================================
Coverage ? 67.94%
=============================================================================================
Files ? 3922
Lines ? 250114
Branches ? 13178
=============================================================================================
Hits ? 169949
Misses ? 64865
Partials ? 15300
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Docs to be added elsewhere |
d955acf
into
45414-software-inventory-vulnerabilities-add-adobe-plugins
Resolves #49060
Ingest fleetd's adobe_plugins table as a new
adobe_pluginssoftware source, so Adobe Creative Cloud CEP and UXP extensions show up in inventory on macOS and Windows hosts.osquery-perf gets -common_adobe_plugins_software_count and -software_adobe_plugins_query_fail_prob.
Related issue: Resolves #
Checklist for submitter
If some of the following don't apply, delete the relevant line.
Testing
Summary by CodeRabbit
New Features
Bug Fixes