• PenguinCoder@beehaw.org
    link
    fedilink
    English
    arrow-up
    0
    ·
    2 months ago

    That query by itself in a vacuum is fine. Combined with many other triggers on the DB, and then federating that out before actually deleting from the local DB… well that is what creates all sorts of headaches.

    • Skull giver@popplesburger.hilciferous.nl
      link
      fedilink
      arrow-up
      0
      ·
      2 months ago

      Images aren’t federated through ActivityPub so I don’t really see how deleting media is supposed to work. Best you can do is delete media for deleted posts.

      The API call for deleting media already exists. It’s actually used in the comment compose screen (you can click the “click here to delete” popup after uploading media). So all you need to do with this info is send a POST request to the existing API endpoint.

      I don’t think the Lemmy database uses triggers on media uploads at all, I don’t think Diesel support those well?

      • Lionir [he/him]@beehaw.org
        link
        fedilink
        arrow-up
        0
        ·
        2 months ago

        Images aren’t federated through ActivityPub so I don’t really see how deleting media is supposed to work.

        Yes, they are. Every instance downloads everyone’s images for a “cached” version that is currently never used. This is what makes this problem especially insidious and straight up dangerous in cases like CSAM.