fix(audit): add missing rows.Err() check after rows.Next() loop #2099

Closed
core-be wants to merge 2 commits from fix/audit-rows-err-check into main
2 changed files with 6 additions and 0 deletions
@@ -252,6 +252,9 @@ func scanAuditRows(rows *sql.Rows) ([]auditEventRow, error) {
}
result = append(result, ev)
}
if err := rows.Err(); err != nil {
return nil, err
}
return result, nil
}
@@ -377,6 +377,9 @@ func readWorkspaceDeriveInputs(ctx context.Context, workspaceID string) (runtime
availableAuthEnv = append(availableAuthEnv, k)
}
}
if err := rows.Err(); err != nil {
log.Printf("llm_billing_mode: rows iteration error for %s: %v (deriving with partial model/auth-env)", workspaceID, err)
}
return runtime, model, availableAuthEnv
}