Package gfit: Unify debug messages for easier reading.

[kraftakt.git] / gfitsync.go

diff --git a/gfitsync.go b/gfitsync.go
index eb3c953..9eadde9 100644 (file)
--- a/gfitsync.go
+++ b/gfitsync.go
@@ -126,8 +126,11 @@ func fitbitGrantHandler(ctx context.Context, w http.ResponseWriter, r *http.Requ
                return err
        }
 
-       if err := c.Subscribe(ctx, "activities"); err != nil {
-               return fmt.Errorf("c.Subscribe() = %v", err)
+       for _, collection := range []string{"activities", "sleep"} {
+               if err := c.Subscribe(ctx, collection); err != nil {
+                       return fmt.Errorf("c.Subscribe(%q) = %v", collection, err)
+               }
+               log.Infof(ctx, "Successfully subscribed to %q", collection)
        }
 
        redirectURL := r.URL
@@ -206,6 +209,7 @@ func handleNotifications(ctx context.Context, payload []byte) error {
 
        for _, s := range subscriptions {
                if s.CollectionType != "activities" {
+                       log.Warningf(ctx, "ignoring collection type %q", s.CollectionType)
                        continue
                }
 
@@ -292,6 +296,8 @@ func handleNotification(ctx context.Context, s *fitbit.Subscription) error {
 
        wg.Add(1)
        go func() {
+               defer wg.Done()
+
                var distanceMeters float64
                for _, d := range summary.Summary.Distances {
                        if d.Activity != "total" {
@@ -302,8 +308,8 @@ func handleNotification(ctx context.Context, s *fitbit.Subscription) error {
                }
                if err := gfitClient.SetDistance(ctx, distanceMeters, tm); err != nil {
                        errs = append(errs, fmt.Errorf("gfitClient.SetDistance(%d) = %v", distanceMeters, err))
+                       return
                }
-               wg.Done()
        }()
 
        wg.Add(1)
@@ -324,7 +330,7 @@ func handleNotification(ctx context.Context, s *fitbit.Subscription) error {
                                continue
                        }
 
-                       startTime, err := time.ParseInLocation("2006-01-02T15:04:05", a.StartDate+"T"+a.StartTime, profile.Timezone)
+                       startTime, err := time.ParseInLocation("2006-01-02T15:04", a.StartDate+"T"+a.StartTime, profile.Timezone)
                        if err != nil {
                                errs = append(errs, fmt.Errorf("gfitClient.SetActivities() = %v", err))
                                return