From 52e5ffaee55f0381efdcc95cab8a0251a1243e63 Mon Sep 17 00:00:00 2001 From: Brad Rydzewski Date: Thu, 7 Aug 2014 21:33:04 -0700 Subject: [PATCH] fixed minor sync error where members were not granted access --- server/handler/login.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/server/handler/login.go b/server/handler/login.go index 12ddfc11f..3043283ae 100644 --- a/server/handler/login.go +++ b/server/handler/login.go @@ -130,7 +130,11 @@ func (h *LoginHandler) GetLogin(w http.ResponseWriter, r *http.Request) error { if err := h.repos.Insert(repo); err != nil { log.Println("Error adding repo.", u.Login, remoteRepo.Name, err) - continue + // this is probably because the repository already exists. + // we should still attempt to grant the user access, however. + // + // todo(bradrydzewski) we should inspect the response to ensure + // the failure is caused by a primary key violation. } // add user permissions