diff --git a/pkg/provider/awn/proxy.go b/pkg/provider/awn/proxy.go index 0bbe466..08456c9 100644 --- a/pkg/provider/awn/proxy.go +++ b/pkg/provider/awn/proxy.go @@ -8,6 +8,7 @@ import ( "gitea.libretechconsulting.com/rmcguire/go-app/pkg/otel" "github.com/go-resty/resty/v2" + "github.com/rs/zerolog/log" "go.opentelemetry.io/otel/attribute" "go.opentelemetry.io/otel/codes" @@ -43,12 +44,18 @@ func (awn *AWNProvider) ProxyReq(ctx context.Context, update *weather.WeatherUpd Get(awnURL) if err != nil { span.RecordError(err) + span.SetAttributes( + attribute.String("query", resp.Request.QueryParam.Encode()), + attribute.String("body", string(resp.Body())), + ) span.SetStatus(codes.Error, err.Error()) + log.Err(err).Any("query", resp.Request.PathParams). + Int("statusCode", resp.StatusCode()). + Msg("awn proxy failed") } span.SetAttributes( attribute.Int("statusCode", resp.StatusCode()), - attribute.String("body", string(resp.Body())), ) return err diff --git a/pkg/provider/wunderground/proxy.go b/pkg/provider/wunderground/proxy.go index 588c117..e1f6d0d 100644 --- a/pkg/provider/wunderground/proxy.go +++ b/pkg/provider/wunderground/proxy.go @@ -6,6 +6,7 @@ import ( "net/url" "github.com/go-resty/resty/v2" + "github.com/rs/zerolog/log" "go.opentelemetry.io/otel/attribute" "go.opentelemetry.io/otel/codes" "k8s.io/utils/ptr" @@ -40,13 +41,17 @@ func (wu *WUProvider) ProxyReq(ctx context.Context, update *weather.WeatherUpdat span.SetStatus(codes.Error, err.Error()) span.SetAttributes( attribute.String("query", resp.Request.QueryParam.Encode()), + attribute.String("body", string(resp.Body())), ) span.RecordError(err) + log.Err(err). + Int("statusCode", resp.StatusCode()). + Any("query", resp.Request.PathParams). + Msg("wunderground proxy failed") } span.SetAttributes( attribute.Int("statusCode", resp.StatusCode()), - attribute.String("body", string(resp.Body())), ) return err