Merge pull request #37 from Questspace-v2/wp/google-username/0

chore: Use random user-number name for google OAuth

src/internal/questspace/authservice/googleservice/google.go

@@ -2,8 +2,13 @@ package googleservice
 
 import (
 	"context"
+	"math/rand"
 	"net/http"
+	"strconv"
+	"time"
 
+	"github.com/cenkalti/backoff/v4"
+	"go.uber.org/zap"
 	"golang.org/x/xerrors"
 	"google.golang.org/api/idtoken"
 
@@ -12,6 +17,7 @@ import (
 	"questspace/internal/questspace/userservice/usertypes"
 	"questspace/pkg/auth/jwt"
 	"questspace/pkg/httperrors"
+	"questspace/pkg/logging"
 	"questspace/pkg/storage"
 )
 
@@ -70,11 +76,12 @@ func (a *Auth) parseToken(ctx context.Context, idToken string) (storage.CreateOr
 	if err != nil {
 		return storage.CreateOrUpdateRequest{}, httperrors.Errorf(http.StatusBadRequest, "bad token: %w", err)
 	}
+	randNum := rand.Int() % (1 << 32) //nolint:gosec
 
 	return storage.CreateOrUpdateRequest{
 		ExternalID: payload.Claims["sub"].(string),
 		CreateUserRequest: storage.CreateUserRequest{
-			Username:  payload.Claims["email"].(string),
+			Username:  "user-" + strconv.Itoa(randNum),
 			AvatarURL: payload.Claims["picture"].(string),
 		},
 	}, nil
@@ -86,7 +93,15 @@ func (a *Auth) doGoogleOAuth(
 	req *storage.CreateOrUpdateRequest,
 	resp *authtypes.Response,
 ) error {
-	user, err := s.CreateOrUpdateByExternalID(ctx, req)
+	policy := backoff.WithContext(
+		backoff.WithMaxRetries(backoff.NewExponentialBackOff(), 5), ctx,
+	)
+
+	user, err := backoff.RetryNotifyWithData(func() (*storage.User, error) {
+		return s.CreateOrUpdateByExternalID(ctx, req)
+	}, policy, func(err error, d time.Duration) {
+		logging.Warn(ctx, "Google OAuth: new retry", zap.Error(err), zap.Duration("passed", d))
+	})
 	if err != nil {
 		return xerrors.Errorf("create or update google user: %w", err)
 	}