Remove gzip exclusion from /sync/devices endpoint now that backend supports it - #9405
Merged
CDRussell merged 1 commit intoAug 6, 2026
Conversation
Member
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
20 tasks
MiSikora
approved these changes
Aug 6, 2026
This was referenced Aug 6, 2026
CDRussell
force-pushed
the
feature/craig/sync_unified_devices_update_device_list_ui_consume_deviceinfo
branch
from
August 6, 2026 14:44
48da81b to
f8bd813
Compare
CDRussell
force-pushed
the
feature/craig/sync_unified_devices_gzip_support_devices_endpoint
branch
from
August 6, 2026 14:44
40717bc to
6be088e
Compare
CDRussell
changed the base branch from
feature/craig/sync_unified_devices_update_device_list_ui_consume_deviceinfo
to
graphite-base/9405
August 6, 2026 14:58
CDRussell
force-pushed
the
feature/craig/sync_unified_devices_gzip_support_devices_endpoint
branch
from
August 6, 2026 14:58
6be088e to
0fcba08
Compare
CDRussell
force-pushed
the
graphite-base/9405
branch
from
August 6, 2026 14:58
f8bd813 to
5ed6d5c
Compare
CDRussell
force-pushed
the
feature/craig/sync_unified_devices_gzip_support_devices_endpoint
branch
from
August 6, 2026 14:59
0fcba08 to
69698ca
Compare
Member
Author
Merge activity
|
CDRussell
deleted the
feature/craig/sync_unified_devices_gzip_support_devices_endpoint
branch
August 6, 2026 15:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Task/Issue URL: https://app.asana.com/1/137249556945/project/72649045549333/task/1217193238272991?focus=true
Tech Design URL (if applicable):
API Proposals URL(s) (if applicable):
Description
The backend now accepts gzipped
PATCHbodies on/sync/devices. When we first integrated with this endpointgzipwasn't supported, soSyncGzipInterceptorexplicitly skipped compression for it. This removes that exclusion so devicesPATCHrequests are compressed like every other syncPATCH.Steps to test this PR
Note
Medium Risk
Changes how device PATCH payloads are sent to the sync API; compression failures still fall back to the uncompressed request via existing error handling.
Overview
SyncGzipInterceptorno longer skips gzip for PATCH requests to/sync/devices. Those calls now follow the same rule as other sync PATCH traffic whengzipPatchRequestsis enabled.The workaround for backend
invalid_jsonon gzipped device PATCH bodies is removed, along with theisDevicesEndpointcheck and related comment.Reviewed by Cursor Bugbot for commit 40717bc. Configure here.