refactor(sdk): add debug logging and reorder address trimming in RD client
- Move port trimming (:80/:443) after name lookup in dialer for correct address handling - Add debug logs for name lookups and lease findings in SDK - Add error logging for relay info retrieval failures
lemon-mint committed
Nov 4, 2025 at 00:26 UTC
3f340827b39d8c551ffb29796befbdafada46d34
2 files changed
+7
-2
cmd/webclient/main_js.go
+4
-2
@@ -33,13 +33,15 @@ var (
33
)
34
35
var rdDialer = func(ctx context.Context, network, address string) (net.Conn, error) {
36
+ address = strings.TrimSuffix(address, ":80")
37
+ address = strings.TrimSuffix(address, ":443")
38
+
39
lease, err := rdClient.LookUpName(address)
40
if err == nil && lease != nil {
41
address = lease.Identity.Id
42
+ log.Debug().Str("name", address).Str("id", lease.Identity.Id).Msg("[SDK] Found lease")
43
}
44
41
- address = strings.TrimSuffix(address, ":80")
42
- address = strings.TrimSuffix(address, ":443")
45
cred := sdk.NewCredential()
46
conn, err := rdClient.Dial(cred, address, "http/1.1")
47
if err != nil {
sdk/sdk.go
+3
@@ -785,6 +785,7 @@ func (g *RDClient) GetRelays() []string {
785
}
786
787
func (g *RDClient) LookUpName(name string) (*rdverb.Lease, error) {
788
+ log.Debug().Str("name", name).Msg("[SDK] Looking up name")
789
var relays []*rdRelay
790
791
g.mu.Lock()
@@ -796,11 +797,13 @@ func (g *RDClient) LookUpName(name string) (*rdverb.Lease, error) {
797
for _, relay := range relays {
798
info, err := relay.client.GetRelayInfo()
799
if err != nil {
800
+ log.Error().Err(err).Str("relay", relay.addr).Msg("[SDK] Error getting relay info")
801
continue
802
}
803
804
for _, lease := range info.Leases {
805
if strings.EqualFold(lease.Name, name) {
806
+ log.Debug().Str("name", name).Str("id", lease.Identity.Id).Msg("[SDK] Found lease")
807
return lease, nil
808
}
809
}