ethyca-fides 2.64.2b0__py2.py3-none-any.whl → 2.64.2rc0__py2.py3-none-any.whl

fides/api/models/manual_tasks/manual_task_config.py

@@ -1,21 +1,11 @@
-from typing import TYPE_CHECKING, Any, Optional, cast
-
-from sqlalchemy import (
-    Boolean,
-    Column,
-    DateTime,
-    ForeignKey,
-    Index,
-    Integer,
-    String,
-    UniqueConstraint,
-)
+from typing import TYPE_CHECKING, Any, Optional
+
+from sqlalchemy import Boolean, Column, ForeignKey, Integer, String
 from sqlalchemy.dialects.postgresql import JSONB
 from sqlalchemy.ext.declarative import declared_attr
 from sqlalchemy.orm import Session, relationship
-from sqlalchemy.sql import func
 
-from fides.api.db.base_class import Base, FidesBase
+from fides.api.db.base_class import Base
 from fides.api.db.util import EnumColumn
 from fides.api.models.manual_tasks.manual_task_log import ManualTaskLog
 from fides.api.schemas.manual_tasks.manual_task_config import (
@@ -23,19 +13,10 @@ from fides.api.schemas.manual_tasks.manual_task_config import (
     ManualTaskFieldMetadata,
     ManualTaskFieldType,
 )
-from fides.api.schemas.manual_tasks.manual_task_schemas import (
-    ManualTaskExecutionTiming,
-    ManualTaskLogStatus,
-)
+from fides.api.schemas.manual_tasks.manual_task_schemas import ManualTaskLogStatus
 
 if TYPE_CHECKING:  # pragma: no cover
     from fides.api.models.manual_tasks.manual_task import ManualTask  # pragma: no cover
-    from fides.api.models.manual_tasks.manual_task_instance import (
-        ManualTaskInstance,  # pragma: no cover
-    )
-    from fides.api.models.manual_tasks.manual_task_instance import (
-        ManualTaskSubmission,  # pragma: no cover
-    )
 
 
 class ManualTaskConfig(Base):
@@ -47,53 +28,13 @@ class ManualTaskConfig(Base):
     def __tablename__(cls) -> str:
         return "manual_task_config"
 
-    # redefined here because there's a minor, unintended discrepancy between
-    # this `id` field and that of the `Base` class, which explicitly sets `index=True`.
-    # TODO: we likely should _not_ be setting `index=True` on the `id`
-    # attribute of the `Base` class, as `primary_key=True` already specifies a
-    # primary key constraint, which will implicitly create an index for the field.
-    id = Column(String(255), primary_key=True, default=FidesBase.generate_uuid)
-    created_at = Column(
-        DateTime(timezone=True), server_default=func.now(), nullable=False
-    )
-    updated_at = Column(
-        DateTime(timezone=True),
-        server_default=func.now(),
-        onupdate=func.now(),
-        nullable=False,
-    )
-
-    task_id = Column(
-        String,
-        ForeignKey(
-            "manual_task.id",
-            name="manual_task_config_task_id_fkey",
-            ondelete="CASCADE",
-        ),
-        nullable=False,
-    )
+    task_id = Column(String, ForeignKey("manual_task.id"), nullable=False)
     config_type = Column(EnumColumn(ManualTaskConfigurationType), nullable=False)
     version = Column(Integer, nullable=False, default=1)
     is_current = Column(Boolean, nullable=False, default=True)
-    execution_timing = Column(
-        EnumColumn(ManualTaskExecutionTiming),
-        nullable=False,
-        default=ManualTaskExecutionTiming.pre_execution,
-    )
-
-    __table_args__ = (
-        Index("ix_manual_task_config_config_type", "config_type"),
-        Index("ix_manual_task_config_task_id", "task_id"),
-    )
 
     # Relationships
-    task = relationship("ManualTask", back_populates="configs", viewonly=True)
-    instances = relationship(
-        "ManualTaskInstance", back_populates="config", uselist=True, viewonly=True
-    )
-    submissions = relationship(
-        "ManualTaskSubmission", back_populates="config", uselist=True, viewonly=True
-    )
+    task = relationship("ManualTask", back_populates="configs")
     field_definitions = relationship(
         "ManualTaskConfigField",
         back_populates="config",
@@ -104,7 +45,7 @@ class ManualTaskConfig(Base):
         "ManualTaskLog",
         back_populates="config",
         primaryjoin="ManualTaskConfig.id == ManualTaskLog.config_id",
-        cascade="all, delete-orphan",
+        viewonly=True,
     )
 
     @classmethod
@@ -148,72 +89,20 @@ class ManualTaskConfigField(Base):
     def __tablename__(cls) -> str:
         return "manual_task_config_field"
 
-    # redefined here because there's a minor, unintended discrepancy between
-    # this `id` field and that of the `Base` class, which explicitly sets `index=True`.
-    # TODO: we likely should _not_ be setting `index=True` on the `id`
-    # attribute of the `Base` class, as `primary_key=True` already specifies a
-    # primary key constraint, which will implicitly create an index for the field.
-    id = Column(String(255), primary_key=True, default=FidesBase.generate_uuid)
-    created_at = Column(
-        DateTime(timezone=True), server_default=func.now(), nullable=False
-    )
-    updated_at = Column(
-        DateTime(timezone=True),
-        server_default=func.now(),
-        onupdate=func.now(),
-        nullable=False,
-    )
-
-    task_id = Column(
-        String,
-        ForeignKey(
-            "manual_task.id",
-            name="manual_task_config_field_task_id_fkey",
-            ondelete="CASCADE",
-        ),
-        nullable=False,
-    )
-    config_id = Column(
-        String,
-        ForeignKey(
-            "manual_task_config.id",
-            name="manual_task_config_field_config_id_fkey",
-            ondelete="CASCADE",
-        ),
-        nullable=False,
-    )
+    task_id = Column(String, ForeignKey("manual_task.id"), nullable=False)
+    config_id = Column(String, ForeignKey("manual_task_config.id"), nullable=False)
     field_key = Column(String, nullable=False)
     field_type = Column(
         EnumColumn(ManualTaskFieldType), nullable=False
     )  # Using ManualTaskFieldType
-    field_metadata: dict[str, Any] = cast(
-        dict[str, Any], Column(JSONB, nullable=False, default={})
-    )
-
-    __table_args__ = (
-        UniqueConstraint("config_id", "field_key", name="unique_field_key_per_config"),
-        Index("ix_manual_task_config_field_config_id", "config_id"),
-        Index("ix_manual_task_config_field_field_key", "field_key"),
-        Index("ix_manual_task_config_field_task_id", "task_id"),
-    )
+    field_metadata = Column(JSONB, nullable=False, default={})
 
     # Relationships
-    config = relationship(
-        "ManualTaskConfig", back_populates="field_definitions", viewonly=True
-    )
-    submissions = relationship(
-        "ManualTaskSubmission",
-        back_populates="field",
-        uselist=True,
-        cascade="all, delete-orphan",
-    )
+    config = relationship("ManualTaskConfig", back_populates="field_definitions")
 
     @property
     def field_metadata_model(self) -> ManualTaskFieldMetadata:
         """Get the field metadata as a Pydantic model."""
-        assert isinstance(
-            self.field_metadata, dict
-        ), "field_metadata must be a dictionary"
         return ManualTaskFieldMetadata.model_validate(self.field_metadata)
 
     @classmethod

fides/api/models/manual_tasks/manual_task_log.py

@@ -1,12 +1,11 @@
 from typing import TYPE_CHECKING, Any, Optional
 
-from sqlalchemy import Column, DateTime, ForeignKey, Index, String
+from sqlalchemy import Column, ForeignKey, String
 from sqlalchemy.dialects.postgresql import JSONB
 from sqlalchemy.ext.declarative import declared_attr
 from sqlalchemy.orm import Session, relationship
-from sqlalchemy.sql import func
 
-from fides.api.db.base_class import Base, FidesBase
+from fides.api.db.base_class import Base
 from fides.api.schemas.manual_tasks.manual_task_schemas import ManualTaskLogStatus
 
 if TYPE_CHECKING:  # pragma: no cover
@@ -14,9 +13,6 @@ if TYPE_CHECKING:  # pragma: no cover
     from fides.api.models.manual_tasks.manual_task_config import (
         ManualTaskConfig,  # pragma: no cover
     )
-    from fides.api.models.manual_tasks.manual_task_instance import (  # pragma: no cover
-        ManualTaskInstance,
-    )
 
 
 class ManualTaskLog(Base):
@@ -27,56 +23,22 @@ class ManualTaskLog(Base):
         """Overriding base class method to set the table name."""
         return "manual_task_log"
 
-    # redefined here because there's a minor, unintended discrepancy between
-    # this `id` field and that of the `Base` class, which explicitly sets `index=True`.
-    # TODO: we likely should _not_ be setting `index=True` on the `id`
-    # attribute of the `Base` class, as `primary_key=True` already specifies a
-    # primary key constraint, which will implicitly create an index for the field.
-    id = Column(String(255), primary_key=True, default=FidesBase.generate_uuid)
-    created_at = Column(
-        DateTime(timezone=True), server_default=func.now(), nullable=False
-    )
-    updated_at = Column(
-        DateTime(timezone=True),
-        server_default=func.now(),
-        onupdate=func.now(),
-        nullable=False,
-    )
-
     task_id = Column(
         String, ForeignKey("manual_task.id", ondelete="CASCADE"), nullable=False
     )
-    config_id = Column(
-        String, ForeignKey("manual_task_config.id", ondelete="CASCADE"), nullable=True
-    )
-    instance_id = Column(
-        String,
-        ForeignKey("manual_task_instance.id", ondelete="CASCADE"),
-        nullable=True,
-    )
+    config_id = Column(String, ForeignKey("manual_task_config.id"), nullable=True)
+    instance_id = Column(String, nullable=True)
     status = Column(String, nullable=False)
-    message = Column(String, nullable=True)
+    message = Column(String, nullable=False)
     details = Column(JSONB, nullable=True)
 
-    __table_args__ = (
-        Index("ix_manual_task_log_config_id", "config_id"),
-        Index("ix_manual_task_log_created_at", "created_at"),
-        Index("ix_manual_task_log_instance_id", "instance_id"),
-        Index("ix_manual_task_log_status", "status"),
-        Index("ix_manual_task_log_task_id", "task_id"),
-    )
-
-    # Relationships
-    task = relationship(
-        "ManualTask", back_populates="logs", foreign_keys=[task_id], viewonly=True
-    )
+    # Relationships - using string references to avoid circular imports
+    task = relationship("ManualTask", back_populates="logs", foreign_keys=[task_id])
     config = relationship(
-        "ManualTaskConfig",
-        back_populates="logs",
-        foreign_keys=[config_id],
-        viewonly=True,
+        "ManualTaskConfig", back_populates="logs", foreign_keys=[config_id]
     )
-    instance = relationship("ManualTaskInstance", back_populates="logs", viewonly=True)
+    # TODO: Add instance relationship when it is implemented
+    # instance = relationship("ManualTaskInstance", back_populates="logs")
 
     @classmethod
     def create_log(

fides/api/models/tcf_publisher_restrictions.py

@@ -10,7 +10,7 @@ from sqlalchemy.ext.asyncio import AsyncSession
 from sqlalchemy.ext.declarative import declared_attr
 from sqlalchemy.orm import Session, relationship
 
-from fides.api.db.base_class import Base, FidesBase
+from fides.api.db.base_class import Base
 
 if TYPE_CHECKING:
     from fides.api.models.privacy_experience import PrivacyExperienceConfig
@@ -83,13 +83,7 @@ class TCFConfiguration(Base):
     def __tablename__(self) -> str:
         return "tcf_configuration"
 
-    # redefined here because there's a minor, unintended discrepancy between
-    # this `id` field and that of the `Base` class, which explicitly sets `index=True`.
-    # TODO: we likely should _not_ be setting `index=True` on the `id`
-    # attribute of the `Base` class, as `primary_key=True` already specifies a
-    # primary key constraint, which will implicitly create an index for the field.
-    id = Column(String(255), primary_key=True, default=FidesBase.generate_uuid)
-    name = Column(String, nullable=False, unique=True)
+    name = Column(String, nullable=False, index=True, unique=True)
 
     privacy_experience_configs = relationship(
         "PrivacyExperienceConfig",
@@ -109,17 +103,11 @@ class TCFPublisherRestriction(Base):
     def __tablename__(self) -> str:
         return "tcf_publisher_restriction"
 
-    # redefined here because there's a minor, unintended discrepancy between
-    # this `id` field and that of the `Base` class, which explicitly sets `index=True`.
-    # TODO: we likely should _not_ be setting `index=True` on the `id`
-    # attribute of the `Base` class, as `primary_key=True` already specifies a
-    # primary key constraint, which will implicitly create an index for the field.
-    id = Column(String(255), primary_key=True, default=FidesBase.generate_uuid)
-
     tcf_configuration_id = Column(
         String(255),
         ForeignKey("tcf_configuration.id", ondelete="CASCADE"),
         nullable=False,
+        index=True,
     )
     purpose_id = Column(Integer, nullable=False)
     restriction_type = Column(EnumColumn(TCFRestrictionType), nullable=False)
@@ -133,7 +121,7 @@ class TCFPublisherRestriction(Base):
     # can be inferred from the presence of the end_vendor_id.
     range_entries = Column(
         ARRAY(JSONB),
-        nullable=False,
+        nullable=True,
         server_default="{}",
         default=list,
     )

fides/api/schemas/manual_tasks/manual_task_schemas.py

@@ -5,15 +5,6 @@ from typing import Annotated, Any, Optional
 from pydantic import ConfigDict, Field
 
 from fides.api.schemas.base_class import FidesSchema
-from fides.api.schemas.manual_tasks.manual_task_status import StatusType
-
-
-class ManualTaskExecutionTiming(str, Enum):
-    """Enum for when a manual task should be executed in the privacy request DAG."""
-
-    pre_execution = "pre_execution"  # Execute before the main DAG
-    post_execution = "post_execution"  # Execute after the main DAG
-    parallel = "parallel"  # Execute in parallel with the main DAG
 
 
 class ManualTaskType(str, Enum):
@@ -32,13 +23,6 @@ class ManualTaskParentEntityType(str, Enum):
     # Add more parent entity types as needed
 
 
-class ManualTaskEntityType(str, Enum):
-    """Enum for manual task entity types."""
-
-    privacy_request = "privacy_request"
-    # Add more entity types as needed
-
-
 class ManualTaskReferenceType(str, Enum):
     """Enum for manual task reference types."""
 
@@ -62,32 +46,6 @@ class ManualTaskLogStatus(str, Enum):
     awaiting_input = "awaiting_input"
 
 
-class ManualTaskResponse(FidesSchema):
-    """Schema for manual task response."""
-
-    model_config = ConfigDict(extra="forbid")
-
-    id: Annotated[str, Field(..., description="Task ID")]
-    parent_entity_id: Annotated[str, Field(..., description="Parent entity ID")]
-    parent_entity_type: Annotated[
-        ManualTaskParentEntityType, Field(..., description="Parent entity type")
-    ]
-    status: Annotated[StatusType, Field(..., description="Task status")]
-    created_at: Annotated[datetime, Field(..., description="Creation timestamp")]
-    updated_at: Annotated[datetime, Field(..., description="Last update timestamp")]
-
-
-class ManualTaskCreate(FidesSchema):
-    """Schema for creating a manual task."""
-
-    model_config = ConfigDict(extra="forbid")
-
-    parent_entity_id: Annotated[str, Field(..., description="Parent entity ID")]
-    parent_entity_type: Annotated[
-        ManualTaskParentEntityType, Field(..., description="Parent entity type")
-    ]
-
-
 class ManualTaskLogCreate(FidesSchema):
     """Schema for creating a manual task log entry."""
 

fides/api/schemas/manual_tasks/manual_task_status.py

@@ -1,6 +1,6 @@
 from datetime import datetime, timezone
 from enum import Enum as EnumType
-from typing import Optional, Protocol
+from typing import Optional
 
 from sqlalchemy.orm import Session
 
@@ -23,133 +23,61 @@ class StatusType(str, EnumType):
 
     @classmethod
     def get_valid_transitions(cls, current_status: "StatusType") -> list["StatusType"]:
-        """Get valid transitions from the current status."""
-        transitions = {
-            cls.pending: [cls.in_progress, cls.failed, cls.completed],
-            cls.in_progress: [cls.completed, cls.failed],
-            cls.completed: [],
-            cls.failed: [cls.pending, cls.in_progress],
-        }
-        return transitions.get(current_status, [])
-
-
-class StatusTransitionProtocol(Protocol):
-    """Protocol for objects that support status transitions.
-
-    This protocol defines the interface that any object supporting status transitions
-    must implement. It includes both the required attributes and methods.
-
-    Example:
-        ```python
-        # Any class that implements this protocol can be used interchangeably
-        def process_status_update(obj: StatusTransitionProtocol, db: Session) -> None:
-            if obj.is_pending:
-                obj.start_progress(db)
-            elif obj.is_in_progress:
-                obj.mark_completed(db, user_id="user123")
-
-        # This works with ManualTaskInstance or any other class implementing the protocol
-        instance = ManualTaskInstance(...)
-        process_status_update(instance, db)
-        ```
-    """
+        """Get valid transitions from the current status.
 
-    # Required attributes - using runtime types that work with SQLAlchemy
-    status: StatusType
-    completed_at: Optional[datetime]  # Can be None when resetting to pending
-    completed_by_id: Optional[str]  # Can be None when resetting to pending
+        Args:
+            current_status: The current status
 
-    # Required methods
-    # pylint does not understand the Protocol abstract syntax and will complain about the ellipsis
-    def update_status(
-        self, db: Session, new_status: StatusType, user_id: Optional[str] = None
-    ) -> None:
-        """Update the status with validation and completion handling."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    def mark_completed(self, db: Session, user_id: str) -> None:
-        """Mark as completed."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    def mark_failed(self, db: Session) -> None:
-        """Mark as failed."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    def start_progress(self, db: Session) -> None:
-        """Mark as in progress."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    def reset_to_pending(self, db: Session) -> None:
-        """Reset to pending status."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    @property
-    def is_completed(self) -> bool:
-        """Check if completed."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    @property
-    def is_failed(self) -> bool:
-        """Check if failed."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    @property
-    def is_in_progress(self) -> bool:
-        """Check if in progress."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-    @property
-    def is_pending(self) -> bool:
-        """Check if pending."""
-        ...  # pylint: disable=unnecessary-ellipsis
-
-
-def validate_status_transition_object(obj: StatusTransitionProtocol) -> bool:
-    """Validate that an object properly implements the StatusTransitionProtocol.
-
-    This function demonstrates how the Protocol can be used for runtime validation
-    and type checking.
-    """
-    required_attrs = ["status", "completed_at", "completed_by_id"]
-    required_methods = [
-        "update_status",
-        "mark_completed",
-        "mark_failed",
-        "start_progress",
-        "reset_to_pending",
-    ]
-    required_properties = ["is_completed", "is_failed", "is_in_progress", "is_pending"]
-
-    # Check all required elements
-    all_required = required_attrs + required_methods + required_properties
-    return all(hasattr(obj, attr) for attr in all_required) and all(
-        callable(getattr(obj, method)) for method in required_methods
-    )
+        Returns:
+            list[StatusType]: List of valid transitions
+        """
+        if current_status == cls.pending:
+            return [cls.in_progress, cls.failed, cls.completed]
+        if current_status == cls.in_progress:
+            return [cls.completed, cls.failed]
+        if current_status == cls.completed:
+            return []
+        if current_status == cls.failed:
+            return [cls.pending, cls.in_progress]
+        return []
 
 
 class StatusTransitionMixin:
     """Mixin for handling status transitions.
 
     This mixin provides methods for managing status transitions and completion tracking.
-    It implements the StatusTransitionProtocol and can be used by any model that needs status management.
+    It can be used by any model that needs status management.
     """
 
-    # Type annotations to match the Protocol
+    # These should be overridden by the implementing class
     status: StatusType
     completed_at: Optional[datetime]
     completed_by_id: Optional[str]
 
     def _get_valid_transitions(self) -> list[StatusType]:
-        """Get valid transitions from the current status."""
+        """Get valid transitions from the current status.
+
+        Returns:
+            list[StatusType]: List of valid transitions
+        """
         return StatusType.get_valid_transitions(self.status)
 
     def _validate_status_transition(self, new_status: StatusType) -> None:
-        """Validate that a status transition is allowed."""
+        """Validate that a status transition is allowed.
+
+        Args:
+            new_status: The new status to transition to
+
+        Raises:
+            StatusTransitionNotAllowed: If the transition is not allowed
+        """
+        # Don't allow transitions to the same status
         if new_status == self.status:
             raise StatusTransitionNotAllowed(
                 f"Invalid status transition: already in status {new_status}"
             )
 
+        # Get valid transitions for current status
         valid_transitions = self._get_valid_transitions()
         if new_status not in valid_transitions:
             raise StatusTransitionNotAllowed(
@@ -160,7 +88,13 @@ class StatusTransitionMixin:
     def update_status(
         self, db: Session, new_status: StatusType, user_id: Optional[str] = None
     ) -> None:
-        """Update the status with validation and completion handling."""
+        """Update the status with validation and completion handling.
+
+        Args:
+            db: Database session
+            new_status: New status to set
+            user_id: Optional user ID who is making the change
+        """
         self._validate_status_transition(new_status)
 
         if new_status == StatusType.completed:
@@ -176,7 +110,12 @@ class StatusTransitionMixin:
         db.commit()
 
     def mark_completed(self, db: Session, user_id: str) -> None:
-        """Mark as completed."""
+        """Mark as completed.
+
+        Args:
+            db: Database session
+            user_id: user ID who completed the task
+        """
         self.update_status(db, StatusType.completed, user_id)
 
     def mark_failed(self, db: Session) -> None:

fides/common/api/v1/urn_registry.py

@@ -155,10 +155,6 @@ AUTHORIZE = "/connection/{connection_key}/authorize"
 ACCESS_MANUAL_WEBHOOKS = "/access_manual_webhook"
 ACCESS_MANUAL_WEBHOOK = CONNECTION_BY_KEY + "/access_manual_webhook"
 
-# Manual Tasks
-MANUAL_TASKS = "/manual-tasks"
-MANUAL_TASK = CONNECTION_BY_KEY + "/manual-task"
-
 # Collection URLs
 DATASETS = "/dataset"
 DATASET_CONFIG = "/datasetconfig"

fides/service/manual_tasks/manual_task_config_service.py

@@ -15,14 +15,6 @@ from fides.api.schemas.manual_tasks.manual_task_config import (
 from fides.service.manual_tasks.utils import validate_fields, with_task_logging
 
 
-class ManualTaskConfigError(Exception):
-    """Exception raised when a manual task config error occurs."""
-
-    def __init__(self, message: str):
-        self.message = message
-        super().__init__(self.message)
-
-
 class ManualTaskConfigService:
     def __init__(self, db: Session):
         self.db = db
@@ -59,8 +51,8 @@ class ManualTaskConfigService:
         """
         try:
             ManualTaskConfigurationType(config_type)
-        except ManualTaskConfigError:
-            raise ManualTaskConfigError(f"Invalid config type: {config_type}")
+        except ValueError:
+            raise ValueError(f"Invalid config type: {config_type}")
 
         # Set all existing versions to non-current
         if is_current:
@@ -104,11 +96,7 @@ class ManualTaskConfigService:
         modified_keys = set(fields_to_remove or [])
 
         if field_updates:
-            try:
-                validate_fields(field_updates, is_submission=False)
-            except ValueError as e:
-                raise ManualTaskConfigError(f"Invalid field updates: {e}") from e
-
+            validate_fields(field_updates, is_submission=False)
             fields_to_create = [
                 {
                     "task_id": config.task_id,
@@ -179,7 +167,7 @@ class ManualTaskConfigService:
         )
 
         if not config:
-            raise ManualTaskConfigError(
+            raise ValueError(
                 f"No current config found for task {task.id} and type {config_type}"
             )
         return config
@@ -367,7 +355,7 @@ class ManualTaskConfigService:
         """
         config = self.db.query(ManualTaskConfig).filter_by(id=config_id).first()
         if not config:
-            raise ManualTaskConfigError(f"Config with ID {config_id} not found")
+            raise ValueError(f"Config with ID {config_id} not found")
 
         log_data = self._create_log_data(
             task.id,