ignore nil repos in list and add better debugging

service/repo/repo.go

@@ -57,8 +57,10 @@ func (s *service) List(ctx context.Context, user *core.User) ([]*core.Repository
 			return nil, err
 		}
 		for _, src := range result {
+			if src != nil {
 				repos = append(repos, convertRepository(src, s.visibility, s.trusted))
 			}
+		}
 		opts.Page = meta.Page.Next
 		opts.URL = meta.Page.NextURL
 

service/repo/repo_test.go

@@ -246,3 +246,44 @@ func TestList_RefreshErr(t *testing.T) {
 		t.Errorf("Expect error refreshing token")
 	}
 }
+
+func TestListWithNilRepo(t *testing.T) {
+	controller := gomock.NewController(t)
+	defer controller.Finish()
+
+	mockUser := &core.User{}
+	mockRepos := []*scm.Repository{
+		{
+			Namespace: "octocat",
+			Name:      "hello-world",
+		},
+		nil,
+	}
+
+	mockRepoService := mockscm.NewMockRepositoryService(controller)
+	mockRepoService.EXPECT().List(gomock.Any(), gomock.Any()).Return(mockRepos, &scm.Response{}, nil)
+
+	mockRenewer := mock.NewMockRenewer(controller)
+	mockRenewer.EXPECT().Renew(gomock.Any(), mockUser, false)
+
+	client := new(scm.Client)
+	client.Repositories = mockRepoService
+
+	want := []*core.Repository{
+		{
+			Namespace:  "octocat",
+			Name:       "hello-world",
+			Slug:       "octocat/hello-world",
+			Visibility: "public",
+		},
+	}
+
+	service := New(client, mockRenewer, "", false)
+	got, err := service.List(noContext, mockUser)
+	if err != nil {
+		t.Error(err)
+	}
+	if diff := cmp.Diff(got, want); diff != "" {
+		t.Errorf(diff)
+	}
+}

service/syncer/syncer.go

@@ -16,6 +16,7 @@ package syncer
 
 import (
 	"context"
+	"runtime/debug"
 	"strings"
 	"time"
 
@@ -67,6 +68,7 @@ func (s *Synchronizer) Sync(ctx context.Context, user *core.User) (*core.Batch,
 		if err := recover(); err != nil {
 			logger = logger.WithField("error", err)
 			logger.Errorln("syncer: unexpected panic")
+			debug.PrintStack()
 		}
 
 		// when the synchronization process is complete