Merge pull request #6 from rarimo/fix/get_provider_panic

Check user is not nil before getting Provider Data

internal/service/api/handlers/get_provider_by_identity_id.go

@@ -21,7 +21,7 @@ func GetProviderByIdentityId(w http.ResponseWriter, r *http.Request) {
 
 	provider, err := KYCService(r).GetProviderByIdentityId(req)
 	switch {
-	case errors.Is(err, identityproviders.ErrProviderNotFound):
+	case errors.Is(err, identityproviders.ErrProviderNotFound), errors.Is(err, identityproviders.ErrUserNotFound):
 		Log(r).WithField("reason", err).Debug("Not found")
 		ape.RenderErr(w, problems.NotFound())
 		return

internal/service/core/handlers.go

@@ -155,6 +155,10 @@ func (k *kycService) GetProviderByIdentityId(req *requests.GetProviderByIdentity
 		return "", errors.Wrap(err, "failed to get user from db with provided identityID")
 	}
 
+	if user == nil {
+		return "", identityproviders.ErrUserNotFound
+	}
+
 	domainData := unstopdom.Domain{}
 	if err := json.Unmarshal(user.ProviderData, &domainData); err != nil {
 		return "", errors.Wrap(err, "failed to unmarshal json")

internal/service/core/identity_providers/models.go

@@ -13,6 +13,7 @@ var (
 
 	// Not found errors
 	ErrProviderNotFound = errors.New("provider not found")
+	ErrUserNotFound     = errors.New("user not found")
 )
 
 type IdentityProviderName string