Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wm => AP: allow posts to be deleted #30

Closed
swentel opened this issue Sep 27, 2018 · 6 comments
Closed

wm => AP: allow posts to be deleted #30

swentel opened this issue Sep 27, 2018 · 6 comments

Comments

@swentel
Copy link
Contributor

swentel commented Sep 27, 2018

Sometimes you might want to delete a post/reply. This is how a minimal delete activity looks like

stdClass Object
(
    [@context] => https://www.w3.org/ns/activitystreams
    [type] => Delete
    [actor] => https://fed.brid.gy/realize.be
    [id] => https://fed.brid.gy/realize.be?unique-id-needed-6
    [to] => Array
        (
            [0] => https://www.w3.org/ns/activitystreams#Public
            [1] => https://mastodon.social/users/swentel
        )

    [object] => stdClass Object
        (
            [id] => https://fed.brid.gy/realize.be?unique-id-needed-6
        )

)

I posted to the shared inbox of mastodon.social in this case. Most of my test replies from yesterday are deleted now.

Not sure how to we'd tell bridgy that though since it works with sending webmentions.

@snarfed
Copy link
Owner

snarfed commented Sep 27, 2018

right! bridgy fed users would trigger a delete by sending a webmention with a source URL that 410s: https://indieweb.org/deleted

@swentel
Copy link
Contributor Author

swentel commented Sep 27, 2018

aha, didn't know that was a valid response too .. :)
I'll have to built this into the drupal module then, good to know!

@barnabywalters
Copy link

It looks like it could be implemented by checking for a 410 source response in https://github.com/snarfed/bridgy-fed/blob/main/webmention.py#L43 and short-circuiting to sending deletion activities rather than looking for a h-entry and converting it to AS, after checking in the datastore to make sure that the thing we’re deleting is something we published in the first place.

snarfed added a commit that referenced this issue Nov 5, 2022
@snarfed
Copy link
Owner

snarfed commented Nov 5, 2022

^ started on this, but it's not quite easy. We store inboxes for every original post we deliver, but not responses (replies, likes, etc), and it takes work and extra requests to distinguish them.

@snarfed
Copy link
Owner

snarfed commented Nov 7, 2022

In progress code is in the delete branch

@snarfed snarfed changed the title Allow posts to be deleted wm => AP: allow posts to be deleted Apr 16, 2023
@snarfed
Copy link
Owner

snarfed commented Apr 17, 2023

Finally implemented this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants