Merge pull request #428 from peppy/general-fixes

Log errors with updating.
ppy · Mar 3, 2017 · 3c714fe · 3c714fe
2 parents 6f3b592 + 9a32322
commit 3c714fe
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/osu.Desktop/Overlays/VersionManager.cs b/osu.Desktop/Overlays/VersionManager.cs
@@ -18,6 +18,7 @@
 using OpenTK;
 using OpenTK.Graphics;
 using System.Net.Http;
+using osu.Framework.Logging;
 
 namespace osu.Desktop.Overlays
 {
@@ -159,15 +160,21 @@ private async void checkForUpdateAsync(bool useDeltaPatching = true, UpdateProgr
 
                     Schedule(() => notification.State = ProgressNotificationState.Completed);
                 }
-                catch (Exception)
+                catch (Exception e)
                 {
                     if (useDeltaPatching)
                     {
+                        Logger.Error(e, @"delta patching failed!");
+
                         //could fail if deltas are unavailable for full update path (https://github.com/Squirrel/Squirrel.Windows/issues/959)
                         //try again without deltas.
                         checkForUpdateAsync(false, notification);
                         scheduleRetry = false;
                     }
+                    else
+                    {
+                        Logger.Error(e, @"update failed!");
+                    }
                 }
             }
             catch (HttpRequestException)