Skip to content

[v17] Prevent WindowsDesktops from exceeding max message size#56155

Merged
rosstimothy merged 1 commit intobranch/v17from
tross/backport-56129/v17
Jun 30, 2025
Merged

[v17] Prevent WindowsDesktops from exceeding max message size#56155
rosstimothy merged 1 commit intobranch/v17from
tross/backport-56129/v17

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

@rosstimothy rosstimothy commented Jun 26, 2025

Backport #56129 to branch/v17

Changelog: Update WindowsDesktop and WindowsDesktopService APIs to use pagination to avoid exceeding message size limitations.

@rosstimothy rosstimothy force-pushed the tross/backport-56129/v17 branch from 84c1efd to 6143e1e Compare June 26, 2025 22:58
- A new ListWindowsDesktops RPC has been added to the auth gRPC
service.

- The cache has been updated to use List instead of GetAll methods
for WindowsDesktops and WindowsDesktopServices.

- The ListWindowsDesktopServices implementation in the cache is now
applying filters if provided.

- tctl get windows_desktop/foo and tctl get windows_desktop_services/foo
have been updated to rely on server side filtering for instead of
attempting client side filtering when retrieving a single resource.

- tctl desktop ls has been updated to rely on listing instead
of GetWindowsDesktops.
@rosstimothy rosstimothy force-pushed the tross/backport-56129/v17 branch from 6143e1e to 9b2f716 Compare June 30, 2025 13:53
@rosstimothy rosstimothy marked this pull request as ready for review June 30, 2025 15:05
@github-actions github-actions Bot added size/md tctl tctl - Teleport admin tool labels Jun 30, 2025
@rosstimothy rosstimothy added this pull request to the merge queue Jun 30, 2025
Merged via the queue into branch/v17 with commit 319f655 Jun 30, 2025
46 checks passed
@rosstimothy rosstimothy deleted the tross/backport-56129/v17 branch June 30, 2025 18:29
@fheinecke fheinecke mentioned this pull request Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/md tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants