fix (SRI-03) : set absolute path url on response instead saving on db

PickHD · Jul 3, 2023 · 47262ad · 47262ad
1 parent e2cc53d
commit 47262ad
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/user/internal/v1/service/user.go b/user/internal/v1/service/user.go
@@ -98,7 +98,7 @@ func (us *UserServiceImpl) GenerateUserShorts(userID string, req *model.ShortUse
 	msg := model.GenerateShortUserMessage{
 		FullURL:  req.FullURL,
 		UserID:   userID,
-		ShortURL: fmt.Sprintf("%s/%s", us.Config.HttpService.ShortenerBaseAPIURL, helper.RandomStringBytesMaskImprSrcSB(8)),
+		ShortURL: helper.RandomStringBytesMaskImprSrcSB(8),
 	}
 
 	err := us.UserRepo.PublishCreateUserShortener(us.Context, &msg)
@@ -107,7 +107,7 @@ func (us *UserServiceImpl) GenerateUserShorts(userID string, req *model.ShortUse
 	}
 
 	return &model.ShortUserResponse{
-		ShortURL: msg.ShortURL,
+		ShortURL: fmt.Sprintf("%s/%s", us.Config.HttpService.ShortenerBaseAPIURL, msg.ShortURL),
 		Method:   "GET",
 	}, nil
 }