fix: retrieving repoid through commit report can yield error (#1132)

Author: JerrySentry

upload/serializers.py

@@ -71,6 +71,7 @@ def create(self, validated_data: Dict[str, Any]) -> ReportSession | None:
         flag_names = (
             validated_data.pop("flags") if "flags" in validated_data.keys() else []
         )
+        repoid = validated_data.pop("repo_id", None)
 
         # default is necessary here, or else if the key is not in the dict
         # the below will throw a KeyError
@@ -81,7 +82,6 @@ def create(self, validated_data: Dict[str, Any]) -> ReportSession | None:
         flags = []
 
         if upload:
-            repoid = upload.report.commit.repository.repoid
             existing_flags_map = self._create_existing_flags_map(repoid)
             for individual_flag in flag_names:
                 flag_obj = existing_flags_map.get(individual_flag, None)

upload/views/uploads.py

@@ -72,7 +72,10 @@ def create_upload(
         serializer.validated_data["storage_path"] = path
     # Create upload record
     instance: ReportSession = serializer.save(
-        report_id=report.id, upload_extras={"format_version": "v1"}, state="started"
+        repo_id=repository.repoid,
+        report_id=report.id,
+        upload_extras={"format_version": "v1"},
+        state="started",
     )
 
     # Inserts mirror upload record into measurements table. CLI hits this endpoint