Skip to content

Commit 164baba

Browse files
committed
refactor Gitaly dialOpts
1 parent 51fe7f2 commit 164baba

File tree

3 files changed

+15
-18
lines changed

3 files changed

+15
-18
lines changed

go/internal/handler/handler.go

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@ import (
55
"os/exec"
66
"syscall"
77

8+
"google.golang.org/grpc"
9+
10+
"gitlab.com/gitlab-org/gitaly/auth"
11+
"gitlab.com/gitlab-org/gitaly/client"
812
"gitlab.com/gitlab-org/gitlab-shell/go/internal/config"
913
"gitlab.com/gitlab-org/gitlab-shell/go/internal/logger"
1014
)
@@ -36,3 +40,12 @@ func execCommand(command string, args ...string) error {
3640
args = append([]string{binPath}, args...)
3741
return syscall.Exec(binPath, args, os.Environ())
3842
}
43+
44+
func dialOpts(token string) []grpc.DialOption {
45+
connOpts := client.DefaultDialOpts
46+
if token := os.Getenv("GITALY_TOKEN"); token != "" {
47+
connOpts = append(client.DefaultDialOpts, grpc.WithPerRPCCredentials(gitalyauth.RPCCredentials(token)))
48+
}
49+
50+
return connOpts
51+
}

go/internal/handler/receive_pack.go

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,7 @@ import (
55
"fmt"
66
"os"
77

8-
"google.golang.org/grpc"
9-
108
pb "gitlab.com/gitlab-org/gitaly-proto/go"
11-
"gitlab.com/gitlab-org/gitaly/auth"
129
"gitlab.com/gitlab-org/gitaly/client"
1310
)
1411

@@ -17,12 +14,7 @@ func ReceivePack(gitalyAddress string, request *pb.SSHReceivePackRequest) (int32
1714
return 0, fmt.Errorf("no gitaly_address given")
1815
}
1916

20-
connOpts := client.DefaultDialOpts
21-
if token := os.Getenv("GITALY_TOKEN"); token != "" {
22-
connOpts = append(client.DefaultDialOpts, grpc.WithPerRPCCredentials(gitalyauth.RPCCredentials(token)))
23-
}
24-
25-
conn, err := client.Dial(gitalyAddress, connOpts)
17+
conn, err := client.Dial(gitalyAddress, dialOpts(os.Getenv("GITALY_TOKEN")))
2618
if err != nil {
2719
return 0, err
2820
}

go/internal/handler/upload_pack.go

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,7 @@ import (
55
"fmt"
66
"os"
77

8-
"google.golang.org/grpc"
9-
108
pb "gitlab.com/gitlab-org/gitaly-proto/go"
11-
"gitlab.com/gitlab-org/gitaly/auth"
129
"gitlab.com/gitlab-org/gitaly/client"
1310
)
1411

@@ -17,12 +14,7 @@ func UploadPack(gitalyAddress string, request *pb.SSHUploadPackRequest) (int32,
1714
return 0, fmt.Errorf("no gitaly_address given")
1815
}
1916

20-
connOpts := client.DefaultDialOpts
21-
if token := os.Getenv("GITALY_TOKEN"); token != "" {
22-
connOpts = append(client.DefaultDialOpts, grpc.WithPerRPCCredentials(gitalyauth.RPCCredentials(token)))
23-
}
24-
25-
conn, err := client.Dial(gitalyAddress, connOpts)
17+
conn, err := client.Dial(gitalyAddress, dialOpts(os.Getenv("GITALY_TOKEN")))
2618
if err != nil {
2719
return 0, err
2820
}

0 commit comments

Comments
 (0)