core/internal: fix regex pattern

Author: dapengzhang0

core/src/main/java/io/grpc/internal/ManagedChannelImpl.java

@@ -84,7 +84,8 @@ public final class ManagedChannelImpl extends ManagedChannel implements WithLogI
   // Matching this pattern means the target string is a URI target or at least intended to be one.
   // A URI target must be an absolute hierarchical URI.
   // From RFC 2396: scheme = alpha *( alpha | digit | "+" | "-" | "." )
-  private static final Pattern URI_PATTERN = Pattern.compile("[a-zA-Z][a-zA-Z0-9+-.]*:/.*");
+  @VisibleForTesting
+  static final Pattern URI_PATTERN = Pattern.compile("[a-zA-Z][a-zA-Z0-9+.-]*:/.*");
 
   private static final ClientTransport SHUTDOWN_TRANSPORT =
       new FailingClientTransport(Status.UNAVAILABLE.withDescription("Channel is shutdown"));

core/src/main/java/io/grpc/internal/ManagedClientTransport.java

@@ -70,7 +70,7 @@ public interface ManagedClientTransport extends ClientTransport, WithLogId {
 
   /**
    * Receives notifications for the transport life-cycle events. Implementation does not need to be
-   * thread-safe, so notifications must be properly sychronized externally.
+   * thread-safe, so notifications must be properly synchronized externally.
    */
   interface Listener {
     /**

core/src/test/java/io/grpc/internal/ManagedChannelImplTest.java

@@ -613,6 +613,16 @@ public void firstResolvedServerConnectedThenDisconnected() throws Exception {
     verify(transport2, timeout(1000)).newStream(same(method), same(headers));
   }
 
+  @Test
+  public void uriPattern() {
+    assertTrue(ManagedChannelImpl.URI_PATTERN.matcher("a:/").matches());
+    assertTrue(ManagedChannelImpl.URI_PATTERN.matcher("Z019+-.:/!@ #~ ").matches());
+    assertFalse(ManagedChannelImpl.URI_PATTERN.matcher("a/:").matches()); // "/:" not matched
+    assertFalse(ManagedChannelImpl.URI_PATTERN.matcher("0a:/").matches()); // '0' not matched
+    assertFalse(ManagedChannelImpl.URI_PATTERN.matcher("a,:/").matches()); // ',' not matched
+    assertFalse(ManagedChannelImpl.URI_PATTERN.matcher(" a:/").matches()); // space not matched
+  }
+
   private static class FakeBackoffPolicyProvider implements BackoffPolicy.Provider {
     @Override
     public BackoffPolicy get() {