Skip to content

Commit

Permalink
Upgrader: NuGet Upgrader should use GitHub endpoint for notifications
Browse files Browse the repository at this point in the history
Fixes a bug where NuGet Upgrader no longer checks the GitHub endpoint to
determine whether to show a notification that an upgrade is available. This did
not affect the actual upgrade command, just whether to show a notification.
  • Loading branch information
jamill committed Jun 20, 2019
1 parent 139600c commit 9a67103
Showing 1 changed file with 16 additions and 14 deletions.
30 changes: 16 additions & 14 deletions GVFS/GVFS.Service/ProductUpgradeTimer.cs
Original file line number Diff line number Diff line change
Expand Up @@ -139,20 +139,22 @@ private void TimerCallback(object unusedState)
return;
}
}

errorMessage = string.Format(
"{0}.{1}: Configured Product Upgrader does not support anonymous version queries.",
nameof(ProductUpgradeTimer),
nameof(this.TimerCallback),
errorMessage);

activity.RelatedWarning(
metadata: new EventMetadata(),
message: errorMessage,
keywords: Keywords.Telemetry);

info.RecordHighestAvailableVersion(highestAvailableVersion: null);
return;
else
{
errorMessage = string.Format(
"{0}.{1}: Configured Product Upgrader does not support anonymous version queries.",
nameof(ProductUpgradeTimer),
nameof(this.TimerCallback),
errorMessage);

activity.RelatedWarning(
metadata: new EventMetadata(),
message: errorMessage,
keywords: Keywords.Telemetry);

info.RecordHighestAvailableVersion(highestAvailableVersion: null);
return;
}
}

InstallerPreRunChecker prerunChecker = new InstallerPreRunChecker(this.tracer, string.Empty);
Expand Down

0 comments on commit 9a67103

Please sign in to comment.