git-codereview: improve haveGerritInternalv1.10.0

Use url.Parse and check URL components rather than performing imprecise
string matching. This addresses a bug where git-codereview does not work
when the Git origin ends with a forward slash.

Note that the check for 'github.com' has been removed since the test for
'.googlesource.com' already excludes it.

Change-Id: I083bccdbacf2152cbfddd2407fb20afa47c8e91e
Reviewed-on: https://go-review.googlesource.com/c/review/+/543495
Reviewed-by: Carlos Amedee <carlos@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Joel Sing <joel@sing.id.au>

1 files changed, 7 insertions, 8 deletions

diff --git a/git-codereview/config.go b/git-codereview/config.go
index debeb5e..cc9286c 100644
--- a/git-codereview/config.go
+++ b/git-codereview/config.go
@@ -6,6 +6,7 @@ package main
 
 import (
 	"fmt"
+	"net/url"
 	"os"
 	"path/filepath"
 	"strings"
@@ -59,20 +60,18 @@ func haveGerritInternal(gerrit, origin string) bool {
 	if gerrit != "" {
 		return true
 	}
-	if strings.Contains(origin, "github.com") {
+
+	u, err := url.Parse(origin)
+	if err != nil {
 		return false
 	}
-	if strings.HasPrefix(origin, "sso://") || strings.HasPrefix(origin, "rpc://") {
+	if u.Scheme == "sso" || u.Scheme == "rpc" {
 		return true
 	}
-	if !strings.Contains(origin, "https://") {
-		return false
-	}
-	if strings.Count(origin, "/") != 3 {
+	if u.Scheme != "https" {
 		return false
 	}
-	host := origin[:strings.LastIndex(origin, "/")]
-	return strings.HasSuffix(host, ".googlesource.com")
+	return strings.HasSuffix(u.Host, ".googlesource.com")
 }
 
 func haveGitHub() bool {