Skip to content

[v17] Show app updater in UI#58261

Merged
gzdunek merged 5 commits intobranch/v17from
gzdunek/backport-updater/v17
Sep 2, 2025
Merged

[v17] Show app updater in UI#58261
gzdunek merged 5 commits intobranch/v17from
gzdunek/backport-updater/v17

Conversation

@gzdunek
Copy link
Copy Markdown
Contributor

@gzdunek gzdunek commented Aug 22, 2025

Backport #57491 to branch/v17
Backport #57695 to branch/v17
Backport #57709 to branch/v17
Backport #58343
Backport #58348

changelog: Teleport Connect now supports managed updates
changelog: Teleport Connect now brings focus back from the browser to itself after a successful SSO login

gzdunek and others added 3 commits August 22, 2025 11:01
* Implement `checkForUpdates`, `download`, `cancelDownload`, `changeManagingCluster` and `quitAndInstall`

* Expose new APIs through Electron IPC

* Add app updater context to store updater state

* Add `AppUpdates` dialog

* Switch "Download" button to "Starting Download..." immediately after clicking it

* Clear managing cluster on logout

* Return correct error in `GetClusterVersions`

* Do not prepend `https://` to file URL, DefaultBaseURL already contains it, trim the checksum value before validating it

* Only try to download an update if it's available

* Remove separate `will-quit` handler

* Clarify comment

* Invert condition in `preventInstallingOutdatedUpdates`

* Add comments

* Move `closeAllConnections()` out of try catch

* Add specialized functions to serialize/deserialize errors

* Add tests

* Leave TODO

* Do not wait for `maybeRemoveAppUpdatesManagingCluster` in `useClusterLogout`

* Hide app updater for .tar.gz which is not supported

* Add missing mock

(cherry picked from commit 13aa750)
* Add app updates widget to login dialog

* Adjust compatibility warning to app updates

* Simplify widget error state, do not show full error message

* Make the widget take up 100% width both in login modal an in story

* Fix LocalWithPasswordless story

* Use `alignItems="stretch"`

* Use `SpaceProps` instead of redefining `mx` manually

* Infer type

* Display buttons in the bottom of Alert

* Avoid moving content when scrollbar appears

* Destructure `Promise.all` result

* Add comment

---------

Co-authored-by: Rafał Cieślak <rafal.cieslak@goteleport.com>
(cherry picked from commit c59a4e6)
* Simplify auto updates multi-cluster view

* Improve labels

* Fix tests

(cherry picked from commit b574045)
* Move login functions from ClustersService to useClusterLogin

* Remove no longer used GetResourcesParams

* Remove clusters/types.ts

* Force focus on Connect after successful SSO login

* Update SSO prompt after login but before sync

* Add test for SSO prompts

* Use `Promise.withResolvers`

Co-authored-by: Grzegorz Zdunek <gzdunek@users.noreply.github.com>

* Update wait-for-sync prompt message

---------

Co-authored-by: Grzegorz Zdunek <gzdunek@users.noreply.github.com>
…pdates (#58348)

* Provide the lowest supported app versions for managed updates

* Deserialize errors in the UI code, not in preload

Otherwise, the error name is lost.

* Bump version in tests

* Lint

* Show exact version in error message

* Do not hide UnsupportedVersionError

* Fix margin

* Make alert take 100% width

(cherry picked from commit d7e1352)
@public-teleport-github-review-bot
Copy link
Copy Markdown

@gzdunek - this PR will require admin approval to merge due to its size. Consider breaking it up into a series smaller changes.

@gzdunek gzdunek enabled auto-merge September 2, 2025 16:45
@gzdunek gzdunek requested a review from r0mant September 2, 2025 19:48
@gzdunek gzdunek added this pull request to the merge queue Sep 2, 2025
Merged via the queue into branch/v17 with commit d8cd3b4 Sep 2, 2025
42 checks passed
@gzdunek gzdunek deleted the gzdunek/backport-updater/v17 branch September 2, 2025 20:50
@fheinecke fheinecke mentioned this pull request Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants