Update exception and documentation links to be human readable (#1259)

grpc · Apr 7, 2021 · 7f34166 · 7f34166
1 parent 78306e4
commit 7f34166
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 8 deletions.
diff --git a/src/Grpc.Net.Client/GrpcChannelOptions.cs b/src/Grpc.Net.Client/GrpcChannelOptions.cs
@@ -43,7 +43,7 @@ public sealed class GrpcChannelOptions
         /// <para>
         /// The underlying <see cref="System.Net.Http.HttpClient"/> used by the channel automatically loads root certificates
         /// from the operating system certificate store.
-        /// Client certificates should be configured on HttpClient. See https://aka.ms/AA6we64 for details.
+        /// Client certificates should be configured on HttpClient. See https://aka.ms/aspnet/grpc/certauth for details.
         /// </para>
         /// </remarks>
         public ChannelCredentials? Credentials { get; set; }

diff --git a/src/Grpc.Net.Client/Internal/DefaultChannelCredentialsConfigurator.cs b/src/Grpc.Net.Client/Internal/DefaultChannelCredentialsConfigurator.cs
@@ -53,7 +53,7 @@ public override void SetSslCredentials(object state, string rootCertificates, Ke
                 throw new InvalidOperationException(
                     $"{nameof(SslCredentials)} with non-null arguments is not supported by {nameof(GrpcChannel)}. " +
                     $"{nameof(GrpcChannel)} uses HttpClient to make gRPC calls and HttpClient automatically loads root certificates from the operating system certificate store. " +
-                    $"Client certificates should be configured on HttpClient. See https://aka.ms/AA6we64 for details.");
+                    $"Client certificates should be configured on HttpClient. See https://aka.ms/aspnet/grpc/certauth for details.");
             }
 
             IsSecure = true;

diff --git a/src/Grpc.Net.Client/Internal/GrpcProtocolHelpers.cs b/src/Grpc.Net.Client/Internal/GrpcProtocolHelpers.cs
@@ -342,7 +342,7 @@ public static Status GetResponseStatus(HttpResponseMessage httpResponse, bool is
                     }
                     if (isWinHttp)
                     {
-                        detail += " Using gRPC with WinHttp has Windows and package version requirements. See https://aka.ms/pzkMXDs for details.";
+                        detail += " Using gRPC with WinHttp has Windows and package version requirements. See https://aka.ms/aspnet/grpc/netstandard for details.";
                     }
 
                     status = new Status(StatusCode.Cancelled, detail);

diff --git a/src/Shared/HttpHandlerFactory.cs b/src/Shared/HttpHandlerFactory.cs
@@ -46,7 +46,7 @@ public static HttpMessageHandler CreatePrimaryHandler()
                 $"gRPC requires extra configuration on .NET implementations that don't support gRPC over HTTP/2. " +
                 $"An HTTP provider must be specified using {nameof(GrpcChannelOptions)}.{nameof(GrpcChannelOptions.HttpHandler)}." +
                 $"The configured HTTP provider must either support HTTP/2 or be configured to use gRPC-Web. " +
-                $"See https://aka.ms/pzkMXDs for details.";
+                $"See https://aka.ms/aspnet/grpc/netstandard for details.";
             throw new PlatformNotSupportedException(message);
 #endif
         }

diff --git a/test/Grpc.Net.Client.Tests/GetTrailersTests.cs b/test/Grpc.Net.Client.Tests/GetTrailersTests.cs
@@ -234,7 +234,7 @@ public async Task AsyncUnaryCall_NoTrailers_WinHttpHandler_ThrowsError()
 
             // Assert
             Assert.AreEqual(StatusCode.Cancelled, ex.StatusCode);
-            Assert.AreEqual("No grpc-status found on response. Using gRPC with WinHttp has Windows and package version requirements. See https://aka.ms/pzkMXDs for details.", ex.Status.Detail);
+            Assert.AreEqual("No grpc-status found on response. Using gRPC with WinHttp has Windows and package version requirements. See https://aka.ms/aspnet/grpc/netstandard for details.", ex.Status.Detail);
             Assert.AreEqual(0, ex.Trailers.Count);
         }
 #endif

diff --git a/test/Grpc.Net.Client.Tests/GrpcChannelTests.cs b/test/Grpc.Net.Client.Tests/GrpcChannelTests.cs
@@ -109,7 +109,7 @@ public void Build_SslCredentialsWithArgs_ThrowsError()
             Assert.AreEqual(
                 "SslCredentials with non-null arguments is not supported by GrpcChannel. " +
                 "GrpcChannel uses HttpClient to make gRPC calls and HttpClient automatically loads root certificates from the operating system certificate store. " +
-                "Client certificates should be configured on HttpClient. See https://aka.ms/AA6we64 for details.", ex.Message);
+                "Client certificates should be configured on HttpClient. See https://aka.ms/aspnet/grpc/certauth for details.", ex.Message);
         }
 
         [Test]
@@ -207,7 +207,7 @@ public void Build_NoHttpProviderOnNetFx_Throw()
                 $"gRPC requires extra configuration on .NET implementations that don't support gRPC over HTTP/2. " +
                 $"An HTTP provider must be specified using {nameof(GrpcChannelOptions)}.{nameof(GrpcChannelOptions.HttpHandler)}." +
                 $"The configured HTTP provider must either support HTTP/2 or be configured to use gRPC-Web. " +
-                $"See https://aka.ms/pzkMXDs for details.";
+                $"See https://aka.ms/aspnet/grpc/netstandard for details.";
 
             Assert.AreEqual(message, ex.Message);
         }

diff --git a/test/Grpc.Net.ClientFactory.Tests/DefaultGrpcClientFactoryTests.cs b/test/Grpc.Net.ClientFactory.Tests/DefaultGrpcClientFactoryTests.cs
@@ -192,7 +192,7 @@ public void CreateClient_NoPrimaryHandlerNetStandard_ThrowError()
             var ex = Assert.Throws<PlatformNotSupportedException>(() => clientFactory.CreateClient<TestGreeterClient>(nameof(TestGreeterClient)))!;
 
             // Assert
-            Assert.AreEqual(@"gRPC requires extra configuration on .NET implementations that don't support gRPC over HTTP/2. An HTTP provider must be specified using GrpcChannelOptions.HttpHandler.The configured HTTP provider must either support HTTP/2 or be configured to use gRPC-Web. See https://aka.ms/pzkMXDs for details.", ex.Message);
+            Assert.AreEqual(@"gRPC requires extra configuration on .NET implementations that don't support gRPC over HTTP/2. An HTTP provider must be specified using GrpcChannelOptions.HttpHandler.The configured HTTP provider must either support HTTP/2 or be configured to use gRPC-Web. See https://aka.ms/aspnet/grpc/netstandard for details.", ex.Message);
         }
 #endif