Skip to content

Commit d3fdf8b

Browse files
authored
Merge pull request #422 from yarpc/dev
Release v0.24.0
2 parents 915ecc3 + c16a5d2 commit d3fdf8b

4 files changed

Lines changed: 11 additions & 5 deletions

File tree

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
Changelog
22
=========
33

4+
# 0.24.0 (2025-01-27)
5+
* Fix add default authority as service name for reflection call
6+
* Added arm64 builds to release
7+
48
# 0.23.0 (2024-06-13)
59
* Introduce new options --force-jaeger-sample to ensure yab requests are traced.
610

protobuf/source_reflection.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,15 @@ func NewDescriptorProviderReflection(args ReflectionArgs) (DescriptorProvider, e
4343
}
4444
r.InitialState(resolver.State{Addresses: peers})
4545

46-
conn, err := grpc.DialContext(context.Background(),
46+
ctx, cancel := context.WithTimeout(context.Background(), args.Timeout)
47+
conn, err := grpc.DialContext(
48+
ctx,
4749
r.Scheme()+":///", // minimal target to dial registered host:port pairs
48-
grpc.WithTimeout(args.Timeout),
50+
grpc.WithAuthority(args.Service),
4951
grpc.WithBlock(),
5052
grpc.WithInsecure())
5153
if err != nil {
54+
cancel()
5255
return nil, fmt.Errorf("could not reach reflection server: %s", err)
5356
}
5457
pbClient := rpb.NewServerReflectionClient(conn)
@@ -65,7 +68,6 @@ func NewDescriptorProviderReflection(args ReflectionArgs) (DescriptorProvider, e
6568
routingHeaders.Append(ygrpc.RoutingKeyHeader, args.RoutingKey)
6669
}
6770

68-
ctx, cancel := context.WithTimeout(context.Background(), args.Timeout)
6971
metadataContext := metadata.NewOutgoingContext(ctx, routingHeaders)
7072
return &grpcreflectSource{
7173
client: grpcreflect.NewClient(metadataContext, pbClient),

scripts/cross_build.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ if [ -z "$VERSION" ]; then
77
fi
88

99
OSs=(linux darwin)
10-
ARCHs=(amd64)
10+
ARCHs=(amd64 arm64)
1111

1212
for OS in "${OSs[@]}"; do
1313
for ARCH in "${ARCHs[@]}"; do

version.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,4 @@ package main
2222

2323
// versionString is the sem-ver version string for yab.
2424
// It will be bumped explicitly on releases.
25-
var versionString = "0.23.0"
25+
var versionString = "0.24.0"

0 commit comments

Comments
 (0)