GODT-2122: Handle check for updates failure

internal/bridge/bridge.go

@@ -365,7 +365,7 @@ func (bridge *Bridge) init(tlsReporter TLSReporter) error {
 
 		version, err := bridge.updater.GetVersionInfo(ctx, bridge.api, bridge.vault.GetUpdateChannel())
 		if err != nil {
-			logrus.WithError(err).Error("Failed to check for updates")
+			bridge.publish(events.UpdateCheckFailed{Error: err})
 		} else {
 			bridge.handleUpdate(version)
 		}

internal/events/update.go

@@ -111,3 +111,14 @@ type UpdateForced struct {
 func (event UpdateForced) String() string {
 	return "UpdateForced"
 }
+
+// UpdateCheckFailed is published when the update check fails.
+type UpdateCheckFailed struct {
+	eventBase
+
+	Error error
+}
+
+func (event UpdateCheckFailed) String() string {
+	return fmt.Sprintf("UpdateCheckFailed: Error: %s", event.Error)
+}

internal/frontend/grpc/service_methods.go

@@ -504,17 +504,24 @@ func (s *Service) CheckUpdate(context.Context, *emptypb.Empty) (*emptypb.Empty,
 	go func() {
 		defer s.panicHandler.HandlePanic()
 
-		updateCh, done := s.bridge.GetEvents(events.UpdateAvailable{}, events.UpdateNotAvailable{})
+		updateCh, done := s.bridge.GetEvents(
+			events.UpdateAvailable{},
+			events.UpdateNotAvailable{},
+			events.UpdateCheckFailed{},
+		)
 		defer done()
 
 		s.bridge.CheckForUpdates()
 
 		switch (<-updateCh).(type) {
+		case events.UpdateNotAvailable:
+			_ = s.SendEvent(NewUpdateIsLatestVersionEvent())
+
 		case events.UpdateAvailable:
 			// ... this is handled by the main event loop
 
-		case events.UpdateNotAvailable:
-			_ = s.SendEvent(NewUpdateIsLatestVersionEvent())
+		case events.UpdateCheckFailed:
+			// ... maybe show an error? but do nothing for now
 		}
 
 		_ = s.SendEvent(NewUpdateCheckFinishedEvent())