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

Add CPE translation mapping for IntelliJ CE for Windows #25971

Merged
merged 2 commits into from
Feb 5, 2025
Merged

Conversation

iansltx
Copy link
Member

@iansltx iansltx commented Feb 2, 2025

Won't solve the false positive issues due to version number mismatches, but will fix the false negative where CE wasn't matching at all, and this is a full fix for IJ CE installed via JetBrains Toolbox.

For #25662.

Checklist for submitter

If some of the following don't apply, delete the relevant line.

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    See Changes files for more information.
  • Added/updated automated tests
  • A detailed QA plan exists on the associated ticket (if it isn't there, work with the product group's QA engineer to add it)
  • Manual QA for all new/changed functionality

Won't solve the false positive issues due to version number mismatches, but will fix the false negative where CE wasn't matching at all.

For #25662.
@iansltx iansltx requested a review from a team as a code owner February 2, 2025 06:17
@iansltx
Copy link
Member Author

iansltx commented Feb 2, 2025

@jmwatts another QA-on-branch PR

Copy link

codecov bot commented Feb 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.61%. Comparing base (ac03358) to head (01be07c).
Report is 173 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #25971      +/-   ##
==========================================
- Coverage   63.61%   63.61%   -0.01%     
==========================================
  Files        1623     1624       +1     
  Lines      155440   155702     +262     
  Branches     4008     4008              
==========================================
+ Hits        98883    99047     +164     
- Misses      48756    48838      +82     
- Partials     7801     7817      +16     
Flag Coverage Δ
backend 64.46% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@iansltx iansltx assigned jmwatts and unassigned ksykulev Feb 5, 2025
@iansltx
Copy link
Member Author

iansltx commented Feb 5, 2025

Assigned for on-branch QA since this affects the vuln feed after merge to main.

Copy link
Member

@jmwatts jmwatts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested fix, works great!

@iansltx iansltx merged commit 136e5f8 into main Feb 5, 2025
32 checks passed
@iansltx iansltx deleted the 25662-ij-windows branch February 5, 2025 23:07
iansltx added a commit that referenced this pull request Feb 5, 2025
Won't solve the false positive issues due to version number mismatches,
but will fix the false negative where CE wasn't matching at all, and
this is a full fix for IJ CE installed via JetBrains Toolbox.

For #25662.

# Checklist for submitter

If some of the following don't apply, delete the relevant line.

<!-- Note that API documentation changes are now addressed by the
product design team. -->

- [x] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
See [Changes
files](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/Committing-Changes.md#changes-files)
for more information.
- [x] Added/updated automated tests
- [x] A detailed QA plan exists on the associated ticket (if it isn't
there, work with the product group's QA engineer to add it)
- [x] Manual QA for all new/changed functionality
iansltx added a commit that referenced this pull request Feb 5, 2025
Won't solve the false positive issues due to version number mismatches,
but will fix the false negative where CE wasn't matching at all, and
this is a full fix for IJ CE installed via JetBrains Toolbox.

For #25662.

# Checklist for submitter

If some of the following don't apply, delete the relevant line.

<!-- Note that API documentation changes are now addressed by the
product design team. -->

- [x] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
See [Changes
files](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/Committing-Changes.md#changes-files)
for more information.
- [x] Added/updated automated tests
- [x] A detailed QA plan exists on the associated ticket (if it isn't
there, work with the product group's QA engineer to add it)
- [x] Manual QA for all new/changed functionality
iansltx added a commit that referenced this pull request Feb 12, 2025
…26116)

Merged into `main` as #25971. PR to RC to get the changelog in the right
place, as this change goes out with the vulns feed update at midnight
UTC.
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.

3 participants