Skip to content

Commit

Permalink
client: fix handling of GRPC_GO_REQUIRE_HANDSHAKE=on to not interpret…
Browse files Browse the repository at this point in the history
… as "hybrid" (#2603)

6cc789b made `envconfig.RequireHandshakeOn` the default when unspecified by environment variable, but missed a fallthrough leading to `GRPC_GO_REQUIRE_HANDSHAKE=on` specifying `envconfig.RequireHandshakeHybrid`.  This change adds the missing fallthrough.
  • Loading branch information
dntj authored and dfawley committed Jan 29, 2019
1 parent 17f2449 commit eda3b7e
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions internal/envconfig/envconfig.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ var (
func init() {
switch strings.ToLower(os.Getenv(requireHandshakeStr)) {
case "on":
fallthrough
default:
RequireHandshake = RequireHandshakeOn
case "off":
Expand Down

0 comments on commit eda3b7e

Please sign in to comment.