ethyca-fides 2.57.2b0__tar.gz → 2.58.1b0__tar.gz

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ethyca_fides
-Version: 2.57.2b0
+Version: 2.58.1b0
 Summary: Open-source ecosystem for data privacy as code.
 Home-page: https://github.com/ethyca/fides
 Author: Ethyca, Inc.

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/ethyca_fides.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ethyca-fides
-Version: 2.57.2b0
+Version: 2.58.1b0
 Summary: Open-source ecosystem for data privacy as code.
 Home-page: https://github.com/ethyca/fides
 Author: Ethyca, Inc.

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/ethyca_fides.egg-info/SOURCES.txt

@@ -820,6 +820,7 @@ src/fides/ui-build/static/admin/_next/static/chunks/3855-b6b7865dedd7bc2a.js
 src/fides/ui-build/static/admin/_next/static/chunks/3872-4e053c20d546f027.js
 src/fides/ui-build/static/admin/_next/static/chunks/4060-8d165e1236ea521a.js
 src/fides/ui-build/static/admin/_next/static/chunks/4294-5b3aa9d605217915.js
+src/fides/ui-build/static/admin/_next/static/chunks/4450-36234280bee624ff.js
 src/fides/ui-build/static/admin/_next/static/chunks/4481-aab99ff80f707473.js
 src/fides/ui-build/static/admin/_next/static/chunks/4723-0a3c5e2ce143a7d0.js
 src/fides/ui-build/static/admin/_next/static/chunks/5258-0658dc2274df6832.js
@@ -829,7 +830,6 @@ src/fides/ui-build/static/admin/_next/static/chunks/5487-3ad50d21cdbc9209.js
 src/fides/ui-build/static/admin/_next/static/chunks/570-c99f07161bd339cd.js
 src/fides/ui-build/static/admin/_next/static/chunks/5826-e5dcb4e68cfe6289.js
 src/fides/ui-build/static/admin/_next/static/chunks/5973-52aee296edc44f7e.js
-src/fides/ui-build/static/admin/_next/static/chunks/605-e692e9cfea2e2cb8.js
 src/fides/ui-build/static/admin/_next/static/chunks/6060-cb1ab5be7067bf7b.js
 src/fides/ui-build/static/admin/_next/static/chunks/618f8807-d15c33d1d6c8b961.js
 src/fides/ui-build/static/admin/_next/static/chunks/6315-fa1519cdf080f42d.js
@@ -841,7 +841,7 @@ src/fides/ui-build/static/admin/_next/static/chunks/6853-8941824350c3c1a8.js
 src/fides/ui-build/static/admin/_next/static/chunks/6954-3b887fb444f9228c.js
 src/fides/ui-build/static/admin/_next/static/chunks/7062.fda15dcb7df85675.js
 src/fides/ui-build/static/admin/_next/static/chunks/7088.eb76fca286e5f2e6.js
-src/fides/ui-build/static/admin/_next/static/chunks/7434-e6fc02c2dd852991.js
+src/fides/ui-build/static/admin/_next/static/chunks/7453-39761c38da31257e.js
 src/fides/ui-build/static/admin/_next/static/chunks/7751-a8f31c062d4cb09d.js
 src/fides/ui-build/static/admin/_next/static/chunks/79-f9b948ebb186900f.js
 src/fides/ui-build/static/admin/_next/static/chunks/7980-4bd08957448dea32.js
@@ -871,7 +871,7 @@ src/fides/ui-build/static/admin/_next/static/chunks/main-app-94a0711202e08b15.js
 src/fides/ui-build/static/admin/_next/static/chunks/polyfills-42372ed130431b0a.js
 src/fides/ui-build/static/admin/_next/static/chunks/webpack-2c7ccac5843c4d8e.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/404-b202c0d8f6fc75c3.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/_app-fa5f8f8b1d6d8c08.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/_app-be7a8cdf3b56b1ec.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/_error-bede3a8288930dcf.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/add-systems-d258f0c25fa020bf.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/ant-poc-b9932971a479f3a7.js
@@ -882,7 +882,7 @@ src/fides/ui-build/static/admin/_next/static/chunks/pages/dataset-fa743ddc7f89d7
 src/fides/ui-build/static/admin/_next/static/chunks/pages/datastore-connection-a78a73b65929853a.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/fides-js-docs-80b241bf6cddb72e.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/index-bfaacdb55a5a6c9f.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/integrations-adb6aced3bdb4899.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/integrations-ef8000d7388dc915.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/login-0cb0eb44bf2dd24c.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/messaging-1e60754abec1ee6b.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/privacy-requests-d85c0d16ba09ba35.js
@@ -890,13 +890,13 @@ src/fides/ui-build/static/admin/_next/static/chunks/pages/properties-20ca2f96390
 src/fides/ui-build/static/admin/_next/static/chunks/pages/systems-abd68fc5ddde5482.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/taxonomy-16b4d75c49276add.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/user-management-6c9ad62479a7d03e.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/add-systems/manual-c2705210ce986109.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/add-systems/manual-9acaab973dfe86e2.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/add-systems/multiple-ba975134a85588f8.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/configure-723cc3d4f5740ea6.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/privacy-experience-c946b33b0322b8ad.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/privacy-notices-ea57f9d6ad17e957.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/properties-e5748812ba055a56.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/reporting-9d2c1f0c9f270d07.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/reporting-788cf0e34829af46.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/configure/add-vendors-b34e5324461d0c87.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/privacy-experience/[id]-cf51752eb0902233.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/consent/privacy-experience/new-be194737b41ccec9.js
@@ -907,12 +907,12 @@ src/fides/ui-build/static/admin/_next/static/chunks/pages/data-catalog/[systemId
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-catalog/[systemId]/projects/[projectUrn]-a29850536c85d4b8.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-catalog/[systemId]/projects/[projectUrn]/[resourceUrn]-ac033a16f043e6f9.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-catalog/[systemId]/resources/[resourceUrn]-71e26d262afcaea5.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/action-center-02439d3aeecfb4b9.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/action-center-d001337d1bb73bd1.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/activity-11b3ce9f61d9bfe9.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/detection-b75dd3e4306ac18e.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/discovery-cad50b0cc6d1050c.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/action-center/[monitorId]-7c74373a4d666a3d.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/action-center/[monitorId]/[systemId]-5182d356672a3b02.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/action-center/[monitorId]-5f9ef1f99818117c.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/action-center/[monitorId]/[systemId]-1a6cff1794122ace.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/detection/[resourceUrn]-054ca46a782e99a5.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/data-discovery/discovery/[resourceUrn]-1da20aeb6fc995e4.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/dataset/[datasetId]-6ba18f92ba561114.js
@@ -941,7 +941,7 @@ src/fides/ui-build/static/admin/_next/static/chunks/pages/settings/email-templat
 src/fides/ui-build/static/admin/_next/static/chunks/pages/settings/locations-66c757325cb58467.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/settings/organization-a08693d0d1e10bc8.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/settings/regulations-c6c239996cfa6ae8.js
-src/fides/ui-build/static/admin/_next/static/chunks/pages/systems/configure/[id]-d3ea9034461511cd.js
+src/fides/ui-build/static/admin/_next/static/chunks/pages/systems/configure/[id]-4f5a28226575c976.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/systems/configure/[id]/test-datasets-151571cff4e85894.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/user-management/new-be690621a944bfe2.js
 src/fides/ui-build/static/admin/_next/static/chunks/pages/user-management/profile/[id]-78eaf933f755bfe8.js
@@ -953,6 +953,8 @@ src/fides/ui-build/static/admin/_next/static/css/95ac327e5d70148d.css
 src/fides/ui-build/static/admin/_next/static/css/972dc7eef106ee7c.css
 src/fides/ui-build/static/admin/_next/static/css/ab65b8cc3144bfc8.css
 src/fides/ui-build/static/admin/_next/static/css/d2d4e9a6923e6730.css
+src/fides/ui-build/static/admin/_next/static/kYmFOkrlCZdjq5U5byh89/_buildManifest.js
+src/fides/ui-build/static/admin/_next/static/kYmFOkrlCZdjq5U5byh89/_ssgManifest.js
 src/fides/ui-build/static/admin/_next/static/media/inter-all-400-normal.2ae8ed37.woff
 src/fides/ui-build/static/admin/_next/static/media/inter-all-500-normal.56c3d87b.woff
 src/fides/ui-build/static/admin/_next/static/media/inter-all-600-normal.d9666969.woff
@@ -986,8 +988,6 @@ src/fides/ui-build/static/admin/_next/static/media/inter-vietnamese-500-normal.7
 src/fides/ui-build/static/admin/_next/static/media/inter-vietnamese-600-normal.eb82de1d.woff2
 src/fides/ui-build/static/admin/_next/static/media/inter-vietnamese-700-normal.00e401a9.woff2
 src/fides/ui-build/static/admin/_next/static/media/logo-white.c9bf4196.svg
-src/fides/ui-build/static/admin/_next/static/sdQ5v46RH3_stSYmL-sZd/_buildManifest.js
-src/fides/ui-build/static/admin/_next/static/sdQ5v46RH3_stSYmL-sZd/_ssgManifest.js
 src/fides/ui-build/static/admin/add-systems/manual.html
 src/fides/ui-build/static/admin/add-systems/multiple.html
 src/fides/ui-build/static/admin/consent/configure.html

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/_version.py

@@ -8,11 +8,11 @@ import json
 
 version_json = '''
 {
- "date": "2025-03-25T14:53:33-0600",
+ "date": "2025-03-26T16:44:40-0600",
  "dirty": true,
  "error": null,
- "full-revisionid": "a5c38571d7f8784361b3b70effecc0a61116196c",
- "version": "2.57.2b0"
+ "full-revisionid": "fcd027892d94a59e6e35a005613442f0063c1137",
+ "version": "2.58.1b0"
 }
 '''  # END VERSION_JSON
 

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/api/graph/traversal.py

@@ -270,6 +270,14 @@ class BaseTraversal:
                 [key.value for key in remaining_node_keys],
             )
 
+        # filter out remaining_edges if the nodes they link are allowed to remain unreachable
+        remaining_edges = {
+            edge
+            for edge in remaining_edges
+            if edge.f1.collection_address() in remaining_node_keys
+            and edge.f2.collection_address() in remaining_node_keys
+        }
+
         # error if there are edges that have not been visited
         if remaining_edges:
             logger.error(

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/api/models/attachment.py

@@ -1,11 +1,13 @@
 import os
 from enum import Enum as EnumType
-from typing import IO, Any, Optional
+from io import BytesIO
+from typing import IO, TYPE_CHECKING, Any, Optional, Tuple, Union
 
+from fideslang.validation import AnyHttpUrlString
 from loguru import logger as log
 from sqlalchemy import Column
 from sqlalchemy import Enum as EnumColumn
-from sqlalchemy import ForeignKey, String, UniqueConstraint
+from sqlalchemy import ForeignKey, String, UniqueConstraint, orm
 from sqlalchemy.ext.declarative import declared_attr
 from sqlalchemy.orm import Session, relationship
 
@@ -23,6 +25,10 @@ from fides.api.service.storage.util import (
     get_local_filename,
 )
 
+if TYPE_CHECKING:
+    from fides.api.models.comment import Comment
+    from fides.api.models.privacy_request import PrivacyRequest
+
 
 class AttachmentType(str, EnumType):
     """
@@ -53,7 +59,9 @@ class AttachmentReference(Base):
         """Overriding base class method to set the table name."""
         return "attachment_reference"
 
-    attachment_id = Column(String, ForeignKey("attachment.id"), nullable=False)
+    attachment_id = Column(
+        String, ForeignKey("attachment.id", ondelete="CASCADE"), nullable=False
+    )
     reference_id = Column(String, nullable=False)
     reference_type = Column(EnumColumn(AttachmentReferenceType), nullable=False)
 
@@ -63,11 +71,8 @@ class AttachmentReference(Base):
         ),
     )
 
-    attachment = relationship(
-        "Attachment",
-        back_populates="references",
-        uselist=False,
-    )
+    # Relationships
+    attachment = relationship("Attachment", back_populates="references")
 
     @classmethod
     def create(
@@ -100,8 +105,11 @@ class Attachment(Base):
     references = relationship(
         "AttachmentReference",
         back_populates="attachment",
-        cascade="all, delete",
+        cascade="all, delete-orphan",
         uselist=True,
+        foreign_keys=[AttachmentReference.attachment_id],
+        primaryjoin=lambda: Attachment.id
+        == orm.foreign(AttachmentReference.attachment_id),
     )
 
     config = relationship(
@@ -127,13 +135,34 @@ class Attachment(Base):
 
         if self.config.type == StorageType.local:
             filename = get_local_filename(self.id)
+
+            # Validate that attachment is a file-like object
+            if not hasattr(attachment, "read"):
+                raise TypeError(f"Expected a file-like object, got {type(attachment)}")
+
+            # Reset the file pointer to the beginning
+            try:
+                attachment.seek(0)
+            except Exception as e:
+                raise ValueError(f"Failed to reset file pointer for attachment: {e}")
+
+            # Write the file in chunks to avoid loading the entire content into memory
             with open(filename, "wb") as file:
-                file.write(attachment.read())
+                for chunk in iter(
+                    lambda: attachment.read(1024 * 1024), b""
+                ):  # 1 MB chunks
+                    if not isinstance(chunk, bytes):
+                        raise TypeError(f"Expected bytes, got {type(chunk)}")
+                    file.write(chunk)
+
+            log.info(f"Uploaded {self.file_name} to local storage at {filename}")
             return
 
         raise ValueError(f"Unsupported storage type: {self.config.type}")
 
-    def retrieve_attachment(self) -> Optional[bytes]:
+    def retrieve_attachment(
+        self,
+    ) -> Optional[Tuple[BytesIO, Union[AnyHttpUrlString, str]]]:
         """Returns the attachment from S3 in bytes form."""
         if self.config.type == StorageType.s3:
             bucket_name = f"{self.config.details[StorageDetails.BUCKET.value]}"
@@ -208,4 +237,44 @@ class Attachment(Base):
     def delete(self, db: Session) -> None:
         """Deletes an attachment record from the database and deletes the attachment from S3."""
         self.delete_attachment_from_storage()
+        for attachment_reference in self.references:
+            attachment_reference.delete(db)
         super().delete(db=db)
+
+    @staticmethod
+    def delete_attachments_for_reference_and_type(
+        db: Session, reference_id: str, reference_type: AttachmentReferenceType
+    ) -> None:
+        """
+        Deletes attachments associated with a given reference_id and reference_type.
+        Deletes all references to the attachments.
+
+        Args:
+            db: Database session
+            reference_id: ID of the reference
+            reference_type: Type of the reference
+
+        Examples:
+
+        - Delete all attachments associated with a comment.
+           ``Attachment.delete_attachments_for_reference_and_type(
+               db, comment.id, AttachmentReferenceType.comment
+           )``
+        - Delete all attachments associated with a privacy request.
+           ``Attachment.delete_attachments_for_reference_and_type(
+               db, privacy_request.id, AttachmentReferenceType.privacy_request
+            )``
+        """
+        # Query attachments explicitly to avoid lazy loading
+        attachments = (
+            db.query(Attachment)
+            .join(AttachmentReference)
+            .filter(
+                AttachmentReference.reference_id == reference_id,
+                AttachmentReference.reference_type == reference_type,
+            )
+            .all()
+        )
+
+        for attachment in attachments:
+            attachment.delete(db)

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/api/models/comment.py

@@ -3,15 +3,17 @@ from typing import TYPE_CHECKING, Any
 
 from sqlalchemy import Column
 from sqlalchemy import Enum as EnumColumn
-from sqlalchemy import ForeignKey, String, UniqueConstraint
+from sqlalchemy import ForeignKey, String, UniqueConstraint, orm
 from sqlalchemy.ext.declarative import declared_attr
 from sqlalchemy.orm import Session, relationship
 
 from fides.api.db.base_class import Base
+from fides.api.models.attachment import Attachment, AttachmentReferenceType
 
 if TYPE_CHECKING:
-    from fides.api.models.attachment import Attachment
+    from fides.api.models.attachment import AttachmentReference
     from fides.api.models.fides_user import FidesUser
+    from fides.api.models.privacy_request import PrivacyRequest
 
 
 class CommentType(str, EnumType):
@@ -87,23 +89,51 @@ class Comment(Base):
     references = relationship(
         "CommentReference",
         back_populates="comment",
-        cascade="all, delete",
-        uselist=True,
-    )
-
-    attachments = relationship(
-        "Attachment",
-        secondary="attachment_reference",
-        primaryjoin="Comment.id == AttachmentReference.reference_id",
-        secondaryjoin="Attachment.id == AttachmentReference.attachment_id",
-        order_by="Attachment.created_at",
+        cascade="all, delete-orphan",
         uselist=True,
+        foreign_keys=[CommentReference.comment_id],
+        primaryjoin=lambda: Comment.id == orm.foreign(CommentReference.comment_id),
     )
 
     def delete(self, db: Session) -> None:
         """Delete the comment and all associated references."""
         # Delete the comment
-        for attachment in self.attachments:
-            if len(attachment.references) == 1:
-                attachment.delete(db)
+        Attachment.delete_attachments_for_reference_and_type(
+            db, self.id, AttachmentReferenceType.comment
+        )
+        for reference in self.references:
+            reference.delete(db)
         db.delete(self)
+
+    @staticmethod
+    def delete_comments_for_reference_and_type(
+        db: Session, reference_id: str, reference_type: CommentReferenceType
+    ) -> None:
+        """
+        Deletes comments associated with a given reference_id and reference_type.
+        Delete all references to the comments.
+
+        Args:
+            db: Database session.
+            reference_id: The reference id to delete.
+            reference_type: The reference type to delete
+
+        Examples:
+          - Delete all comments associated with a privacy request.
+            ``Comment.delete_comments_for_reference_and_type(
+                db, privacy_request.id, CommentReferenceType.privacy_request
+            )``
+        """
+        # Query comments explicitly to avoid lazy loading
+        comments = (
+            db.query(Comment)
+            .join(CommentReference)
+            .filter(
+                CommentReference.reference_id == reference_id,
+                CommentReference.reference_type == reference_type,
+            )
+            .all()
+        )
+
+        for comment in comments:
+            comment.delete(db)

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/api/models/privacy_request/privacy_request.py

@@ -39,10 +39,14 @@ from fides.api.graph.config import (
     CollectionAddress,
 )
 from fides.api.migrations.hash_migration_mixin import HashMigrationMixin
-from fides.api.models.attachment import Attachment, AttachmentReference
+from fides.api.models.attachment import (
+    Attachment,
+    AttachmentReference,
+    AttachmentReferenceType,
+)
 from fides.api.models.audit_log import AuditLog
 from fides.api.models.client import ClientDetail
-from fides.api.models.comment import Comment, CommentReference
+from fides.api.models.comment import Comment, CommentReference, CommentReferenceType
 from fides.api.models.fides_user import FidesUser
 from fides.api.models.manual_webhook import AccessManualWebhook
 from fides.api.models.policy import (
@@ -169,18 +173,25 @@ class PrivacyRequest(
         backref="privacy_requests",
     )
     attachments = relationship(
-        Attachment,
+        "Attachment",
         secondary="attachment_reference",
-        primaryjoin="PrivacyRequest.id == AttachmentReference.reference_id",
+        primaryjoin="and_(PrivacyRequest.id == AttachmentReference.reference_id, "
+        "AttachmentReference.reference_type == 'privacy_request')",
         secondaryjoin="Attachment.id == AttachmentReference.attachment_id",
         order_by="Attachment.created_at",
+        viewonly=True,
+        uselist=True,
     )
+
     comments = relationship(
-        Comment,
+        "Comment",
         secondary="comment_reference",
-        primaryjoin="PrivacyRequest.id == CommentReference.reference_id",
+        primaryjoin="and_(PrivacyRequest.id == CommentReference.reference_id, "
+        "CommentReference.reference_type == 'privacy_request')",
         secondaryjoin="Comment.id == CommentReference.comment_id",
         order_by="Comment.created_at",
+        viewonly=True,
+        uselist=True,
     )
     property_id = Column(String, nullable=True)
 
@@ -323,6 +334,12 @@ class PrivacyRequest(
         deleting this object from the database
         """
         self.clear_cached_values()
+        Attachment.delete_attachments_for_reference_and_type(
+            db, self.id, AttachmentReferenceType.privacy_request
+        )
+        Comment.delete_comments_for_reference_and_type(
+            db, self.id, CommentReferenceType.privacy_request
+        )
 
         for provided_identity in self.provided_identities:  # type: ignore[attr-defined]
             provided_identity.delete(db=db)

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/api/schemas/connection_configuration/connection_secrets_datahub.py

@@ -30,13 +30,11 @@ class DatahubSchema(ConnectionConfigSecretsSchema):
     )
     frequency: PeriodicIntegrationFrequency = Field(
         title="Frequency",
-        description="The frequency at which the integration should run. Defaults to daily.",
-        default=PeriodicIntegrationFrequency.daily,
+        description="The frequency at which the integration should run. Available options are daily, weekly, and monthly.",
     )
     glossary_node: str = Field(
         title="Glossary Node",
-        description="The glossary node name to use on Datahub for Fides Data Categories. Defaults to FidesDataCategories",
-        default="FidesDataCategories",
+        description="The glossary node name to use on Datahub for Fides Data Categories. (e.g. FidesDataCategories)",
     )
 
     _required_components: ClassVar[List[str]] = ["datahub_server_url", "datahub_token"]

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/api/service/storage/s3.py

@@ -64,7 +64,7 @@ def generic_upload_to_s3(  # pylint: disable=R0913
     size_threshold: int = LARGE_FILE_THRESHOLD,  # 5 MB threshold
 ) -> Optional[AnyHttpUrlString]:
     """
-    Uploads arbitrary data to S3 returned from an access request.
+    Uploads file like objects to S3.
     Handles both small and large uploads.
 
     :param storage_secrets: S3 storage secrets
@@ -75,6 +75,19 @@ def generic_upload_to_s3(  # pylint: disable=R0913
     """
     logger.info("Starting S3 Upload of {}", file_key)
 
+    # Validate that the document is a file-like object
+    if not hasattr(document, "read") or not hasattr(document, "seek"):
+        raise TypeError(
+            f"The 'document' parameter must be a file-like object supporting 'read' and 'seek'. "
+            f"Received: {type(document)}"
+        )
+
+    # Ensure the file pointer is at the beginning
+    try:
+        document.seek(0)
+    except Exception as e:
+        raise ValueError(f"Failed to reset file pointer for document: {e}")
+
     s3_client = maybe_get_s3_client(auth_method, storage_secrets)
 
     # Define a transfer configuration for multipart uploads

{ethyca_fides-2.57.2b0 → ethyca_fides-2.58.1b0}/src/fides/ui-build/static/admin/404.html

@@ -1 +1 @@
-<!DOCTYPE html><html lang="en"><head><meta charSet="utf-8"/><meta name="viewport" content="width=device-width"/><meta name="next-head-count" content="2"/><link data-next-font="" rel="preconnect" href="/" crossorigin="anonymous"/><link rel="preload" href="/_next/static/css/113d823fe71f6af0.css" as="style"/><link rel="stylesheet" href="/_next/static/css/113d823fe71f6af0.css" data-n-g=""/><noscript data-n-css=""></noscript><script defer="" nomodule="" src="/_next/static/chunks/polyfills-42372ed130431b0a.js"></script><script src="/_next/static/chunks/webpack-2c7ccac5843c4d8e.js" defer=""></script><script src="/_next/static/chunks/framework-c92fc3344e6fd165.js" defer=""></script><script src="/_next/static/chunks/main-24f422f93845a596.js" defer=""></script><script src="/_next/static/chunks/pages/_app-fa5f8f8b1d6d8c08.js" defer=""></script><script src="/_next/static/chunks/pages/404-b202c0d8f6fc75c3.js" defer=""></script><script src="/_next/static/sdQ5v46RH3_stSYmL-sZd/_buildManifest.js" defer=""></script><script src="/_next/static/sdQ5v46RH3_stSYmL-sZd/_ssgManifest.js" defer=""></script><style>.data-ant-cssinjs-cache-path{content:"";}</style></head><body><div id="__next"><div style="height:100%;display:flex"></div></div><script id="__NEXT_DATA__" type="application/json">{"props":{"pageProps":{}},"page":"/404","query":{},"buildId":"sdQ5v46RH3_stSYmL-sZd","nextExport":true,"autoExport":true,"isFallback":false,"scriptLoader":[]}</script></body></html>
+<!DOCTYPE html><html lang="en"><head><meta charSet="utf-8"/><meta name="viewport" content="width=device-width"/><meta name="next-head-count" content="2"/><link data-next-font="" rel="preconnect" href="/" crossorigin="anonymous"/><link rel="preload" href="/_next/static/css/113d823fe71f6af0.css" as="style"/><link rel="stylesheet" href="/_next/static/css/113d823fe71f6af0.css" data-n-g=""/><noscript data-n-css=""></noscript><script defer="" nomodule="" src="/_next/static/chunks/polyfills-42372ed130431b0a.js"></script><script src="/_next/static/chunks/webpack-2c7ccac5843c4d8e.js" defer=""></script><script src="/_next/static/chunks/framework-c92fc3344e6fd165.js" defer=""></script><script src="/_next/static/chunks/main-24f422f93845a596.js" defer=""></script><script src="/_next/static/chunks/pages/_app-be7a8cdf3b56b1ec.js" defer=""></script><script src="/_next/static/chunks/pages/404-b202c0d8f6fc75c3.js" defer=""></script><script src="/_next/static/kYmFOkrlCZdjq5U5byh89/_buildManifest.js" defer=""></script><script src="/_next/static/kYmFOkrlCZdjq5U5byh89/_ssgManifest.js" defer=""></script><style>.data-ant-cssinjs-cache-path{content:"";}</style></head><body><div id="__next"><div style="height:100%;display:flex"></div></div><script id="__NEXT_DATA__" type="application/json">{"props":{"pageProps":{}},"page":"/404","query":{},"buildId":"kYmFOkrlCZdjq5U5byh89","nextExport":true,"autoExport":true,"isFallback":false,"scriptLoader":[]}</script></body></html>