[analyze] Fix double-print in postgres analyzer (#3199)

* [analyze] Fix double-print in postgres analyzer * Continue on error in github analyzer
trufflesecurity · Aug 7, 2024 · 39f5f54 · 39f5f54
1 parent d0726eb
commit 39f5f54
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/pkg/analyzer/analyzers/github/finegrained/finegrained.go b/pkg/analyzer/analyzers/github/finegrained/finegrained.go
@@ -1240,7 +1240,8 @@ func analyzeUserPermissions(client *gh.Client, user *gh.User) ([]Permission, err
 	for _, permFunc := range acctPermFuncMap {
 		access, err := permFunc(client, user)
 		if err != nil {
-			return nil, err
+			// TODO: Log error.
+			continue
 		}
 		perms = append(perms, access)
 	}

diff --git a/pkg/analyzer/analyzers/postgres/postgres.go b/pkg/analyzer/analyzers/postgres/postgres.go
@@ -86,8 +86,6 @@ func AnalyzeAndPrintPermissions(cfg *config.Config, connectionStr string) {
 	color.Yellow("[!] Successfully connected to Postgres database.")
 	printUserRoleAndPriv(info.Role, info.RolePrivs)
 
-	printDBPrivs(info.DBs, info.User)
-
 	// Print db privs
 	if len(info.DBs) > 0 {
 		fmt.Print("\n\n")