upgraded logging for #595 log url when connection is closed (#44953)

Author: surayya-MS

src/Containers/Microsoft.NET.Build.Containers/ContainerBuilder.cs

@@ -237,6 +237,11 @@ private static async Task<int> PushToRemoteRegistryAsync(ILogger logger, BuiltIm
                 cancellationToken)).ConfigureAwait(false);
             logger.LogInformation(Strings.ContainerBuilder_ImageUploadedToRegistry, destinationImageReference, destinationImageReference.RemoteRegistry.RegistryName);
         }
+        catch (UnableToDownloadFromRepositoryException)
+        {
+            logger.LogError(Resource.FormatString(nameof(Strings.UnableToDownloadFromRepository)), sourceImageReference);
+            return 1;
+        }
         catch (UnableToAccessRepositoryException)
         {
             logger.LogError(Resource.FormatString(nameof(Strings.UnableToAccessRepository), destinationImageReference.Repository, destinationImageReference.RemoteRegistry!.RegistryName));

src/Containers/Microsoft.NET.Build.Containers/Exceptions/UnableToDownloadFromRepositoryException.cs

@@ -0,0 +1,12 @@
+// Copyright (c) .NET Foundation and contributors. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+namespace Microsoft.NET.Build.Containers;
+
+internal sealed class UnableToDownloadFromRepositoryException : Exception
+{
+    public UnableToDownloadFromRepositoryException(string repository)
+        : base($"The download of the image from repository { repository } has failed.")
+    {
+    }
+}

src/Containers/Microsoft.NET.Build.Containers/Registry/Registry.cs

@@ -408,15 +408,22 @@ public async Task<string> DownloadBlobAsync(string repository, Descriptor descri
             return localPath;
         }
 
-        // No local copy, so download one
-        using Stream responseStream = await _registryAPI.Blob.GetStreamAsync(repository, descriptor.Digest, cancellationToken).ConfigureAwait(false);
-
         string tempTarballPath = ContentStore.GetTempFile();
-        using (FileStream fs = File.Create(tempTarballPath))
+
+        try
         {
-            await responseStream.CopyToAsync(fs, cancellationToken).ConfigureAwait(false);
-        }
+            // No local copy, so download one
+            using Stream responseStream = await _registryAPI.Blob.GetStreamAsync(repository, descriptor.Digest, cancellationToken).ConfigureAwait(false);
 
+            using (FileStream fs = File.Create(tempTarballPath))
+            {
+                await responseStream.CopyToAsync(fs, cancellationToken).ConfigureAwait(false);
+            }
+        }
+        catch (Exception)
+        {
+            throw new UnableToDownloadFromRepositoryException(repository);
+        }
         cancellationToken.ThrowIfCancellationRequested();
 
         File.Move(tempTarballPath, localPath, overwrite: true);

src/Containers/Microsoft.NET.Build.Containers/Resources/Strings.Designer.cs

@@ -461,6 +461,10 @@
     <value>CONTAINER1015: Unable to access the repository '{0}' at tag '{1}' in the registry '{2}'. Please confirm that this name and tag are present in the registry.</value>
     <comment>{StrBegin="CONTAINER1015: "}</comment>
   </data>
+  <data name="UnableToDownloadFromRepository" xml:space="preserve">
+    <value>CONTAINER1018: Unable to download image from the repository '{0}'.</value>
+    <comment>{StrBegins="CONTAINER1018:" }</comment>
+  </data>
   <data name="UnableToAccessRepository" xml:space="preserve">
     <value>CONTAINER1016: Unable to access the repository '{0}' in the registry '{1}'. Please confirm your credentials are correct and that you have access to this repository and registry.</value>
     <comment>{StrBegin="CONTAINER1016:" }</comment>