excel export: enhance handling of finding groups, better logging (#12435)

* excel export: enhance error logging

* excel export: fix foreign key errors

Author: valentijnscholten

dojo/reports/views.py

@@ -816,7 +816,7 @@ def get(self, request):
         writer = csv.writer(response)
         allowed_attributes = get_attributes()
         excludes_list = get_excludes()
-        allowed_foreign_keys = get_attributes()
+        allowed_foreign_keys = get_foreign_keys()
         first_row = True
 
         for finding in findings:
@@ -940,10 +940,11 @@ def get(self, request):
         self.font_bold = font_bold
         allowed_attributes = get_attributes()
         excludes_list = get_excludes()
-        allowed_foreign_keys = get_attributes()
+        allowed_foreign_keys = get_foreign_keys()
 
         row_num = 1
         for finding in findings:
+            logger.debug(f"processing finding: {finding.id}")
             if row_num == 1:
                 col_num = 1
                 for key in dir(finding):
@@ -955,7 +956,7 @@ def get(self, request):
                             cell.font = font_bold
                             col_num += 1
                     except Exception as exc:
-                        logger.error("Error in attribute: " + str(exc))
+                        logger.warning(f"Error in attribute: {key}" + str(exc))
                         cell = worksheet.cell(row=row_num, column=col_num, value=key)
                         col_num += 1
                         continue
@@ -1007,7 +1008,7 @@ def get(self, request):
                             worksheet.cell(row=row_num, column=col_num, value=value)
                             col_num += 1
                     except Exception as exc:
-                        logger.error("Error in attribute: " + str(exc))
+                        logger.warning(f"Error in attribute: {key}" + str(exc))
                         worksheet.cell(row=row_num, column=col_num, value="Value not supported")
                         col_num += 1
                         continue