Skip to content

Commit

Permalink
Added Nil Check For Signature Query (#2859)
Browse files Browse the repository at this point in the history
Signed-off-by: David Deal <[email protected]>
  • Loading branch information
dealako authored Apr 7, 2021
1 parent 8eb4740 commit 96f8c50
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cla-backend-go/signatures/repository.go
Original file line number Diff line number Diff line change
Expand Up @@ -1407,12 +1407,12 @@ func (repo repository) GetProjectCompanyEmployeeSignatures(ctx context.Context,
filter := expression.Name("signature_approved").Equal(expression.Value(aws.Bool(true))).
And(expression.Name("signature_signed").Equal(expression.Value(aws.Bool(true))))

if criteria.GitHubUsername != "" {
if criteria != nil && criteria.GitHubUsername != "" {
log.WithFields(f).Debugf("adding Githubusername criteria filter for :%s ", criteria.GitHubUsername)
filter = filter.And(expression.Name("user_github_username").Equal(expression.Value(criteria.GitHubUsername)))
}

if criteria.UserEmail != "" {
if criteria != nil && criteria.UserEmail != "" {
log.WithFields(f).Debugf("adding useremail criteria filter for : %s ", criteria.UserEmail)
filter = filter.And(expression.Name("user_email").Equal(expression.Value(criteria.UserEmail)))
}
Expand Down

0 comments on commit 96f8c50

Please sign in to comment.