Skip to content

Cherrypick 35309 followup into 4.76.1 - #35829

Merged
JordanMontgomery merged 1 commit into
rc-patch-fleet-v4.76.1from
JM-35309-followup-4.76.1
Nov 17, 2025
Merged

Cherrypick 35309 followup into 4.76.1#35829
JordanMontgomery merged 1 commit into
rc-patch-fleet-v4.76.1from
JM-35309-followup-4.76.1

Conversation

@JordanMontgomery

Copy link
Copy Markdown
Member

Related issue: Resolves #35309

Original PR: #35818

Followup changes, see
https://fleetdm.slack.com/archives/C019WG4GH0A/p1763137466439419 for more context. We decided not to use the initially proposed PUT endpoint at all and update the existing POST endpoint to have the desired behavior

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.

  • Input data is properly validated, SELECT * is avoided, SQL injection is prevented (using placeholders for values in statements)

  • If paths of existing endpoints are modified without backwards compatibility, checked the frontend/CLI for any necessary changes

Testing

…cripts (#35818)

<!-- Add the related story/sub-task/bug number, like Resolves #123, or
remove if NA -->
**Related issue:** Resolves #35309 

Followup changes, see
https://fleetdm.slack.com/archives/C019WG4GH0A/p1763137466439419 for
more context. We decided not to use the initially proposed PUT endpoint
at all and update the existing POST endpoint to have the desired
behavior

# Checklist for submitter

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

- [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/guides/committing-changes.md#changes-files)
for more information.

- [x] Input data is properly validated, `SELECT *` is avoided, SQL
injection is prevented (using placeholders for values in statements)
- [x] If paths of existing endpoints are modified without backwards
compatibility, checked the frontend/CLI for any necessary changes

## Testing

- [x] Added/updated automated tests
- [x] Where appropriate, [automated tests simulate multiple hosts and
test for host
isolation](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/reference/patterns-backend.md#unit-testing)
(updates to one hosts's records do not affect another)

- [x] QA'd all new/changed functionality manually
@codecov

codecov Bot commented Nov 17, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (rc-patch-fleet-v4.76.1@3e3003d). Learn more about missing BASE report.

Files with missing lines Patch % Lines
server/datastore/mysql/setup_experience.go 63.63% 2 Missing and 2 partials ⚠️
ee/server/service/setup_experience.go 50.00% 0 Missing and 1 partial ⚠️
server/service/setup_experience.go 66.66% 1 Missing ⚠️
Additional details and impacted files
@@                    Coverage Diff                    @@
##             rc-patch-fleet-v4.76.1   #35829   +/-   ##
=========================================================
  Coverage                          ?   67.54%           
=========================================================
  Files                             ?     1339           
  Lines                             ?   151618           
  Branches                          ?        0           
=========================================================
  Hits                              ?   102416           
  Misses                            ?    38742           
  Partials                          ?    10460           
Flag Coverage Δ
backend 67.54% <68.42%> (?)

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JordanMontgomery

Copy link
Copy Markdown
Member Author

Test failure appears to be unrelated and is one we've seen randomly fail before.

@JordanMontgomery
JordanMontgomery merged commit 7b55bdf into rc-patch-fleet-v4.76.1 Nov 17, 2025
39 of 41 checks passed
@JordanMontgomery
JordanMontgomery deleted the JM-35309-followup-4.76.1 branch November 17, 2025 16:48
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