Added more connection timeout logic to GetPackage

NuGet · Feb 3, 2014 · 0926f47 · 0926f47
1 parent 563cf10
commit 0926f47
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/NuGetGallery/Controllers/ApiController.cs b/src/NuGetGallery/Controllers/ApiController.cs
@@ -96,6 +96,9 @@ public virtual async Task<ActionResult> GetPackage(string id, string version)
             // The allow prerelease flag is ignored if both partialId and version are specified.
             // In general we want to try to add download statistics for any package regardless of whether a version was specified.
 
+            // Only allow database requests to take up to 5 seconds.  Any longer and we just lose the data; oh well.
+            EntitiesContext.SetCommandTimeout(5);
+
             Package package = null;
             try
             {
@@ -118,9 +121,6 @@ public virtual async Task<ActionResult> GetPackage(string id, string version)
                         ProjectGuids = Request.Headers["NuGet-ProjectGuids"],
                     };
 
-                    // Only allow this to take up to 5 seconds.  Any longer and we just lose the data; oh well.
-                    EntitiesContext.SetCommandTimeout(5);
-
                     PackageService.AddDownloadStatistics(stats);
                 }
                 catch (ReadOnlyModeException)