refactor: default direction should be asc

Co-authored-by: ybelmekk <[email protected]>

Author: tommytroen

Diff for: internal/api/grpcvulnerabilities/vulnerabilities.go

@@ -100,7 +100,7 @@ func (s *Server) ListVulnerabilitiesForImage(ctx context.Context, request *vulne
 		IncludeSuppressed: &request.IncludeSuppressed,
 		Offset:            offset,
 		Limit:             limit,
-		OrderBy:           sanitizeOrderBy(request.OrderBy, ""),
+		OrderBy:           sanitizeOrderBy(request.OrderBy, vulnerabilities.OrderBySeverity),
 	})
 
 	if err != nil {
@@ -246,7 +246,7 @@ func sanitizeOrderBy(orderBy *vulnerabilities.OrderBy, defaultOrder vulnerabilit
 	if orderBy == nil {
 		orderBy = &vulnerabilities.OrderBy{
 			Field:     string(defaultOrder),
-			Direction: vulnerabilities.Direction_DESC,
+			Direction: vulnerabilities.Direction_ASC,
 		}
 	}
 

Diff for: pkg/api/vulnerabilities/options.go

@@ -1,7 +1,6 @@
 package vulnerabilities
 
 import (
-	"fmt"
 	"google.golang.org/grpc"
 )
 
@@ -138,7 +137,6 @@ func Offset(offset int32) Option {
 }
 
 func Order(field OrderByField, direction Direction) Option {
-	fmt.Println(field)
 	return newFuncOption(func(o *options) {
 		o.orderBy = &OrderBy{
 			Field:     string(field),