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

Planetary will not download your own feed #847

Closed
quickdudley opened this issue Sep 17, 2022 · 11 comments
Closed

Planetary will not download your own feed #847

quickdudley opened this issue Sep 17, 2022 · 11 comments
Assignees
Labels
bug Something isn't working

Comments

@quickdudley
Copy link

To replicate:

  1. Reinstall planetary on an iOS device which has previously had it installed and go through the mandatory new account setup
  2. In the advanced and dangerous settings: select the previous account

Expected results:
Planetary should redownload the original feed

Actual results:
Planetary followed the account @l/lzpaG3asUP+kSj1KNdEvh+oIzjmsPBk20X5er3YrI=.ed25519 and did not download any messages from the feed being restored.

@mplorentz
Copy link
Member

I was able to reproduce this today.

As I explained on SSB, the Planetary Help auto-follow does not actually publish a message on your feed and fork it, although the UI does make it seem that way. I opened #873 to make this more obvious.

The real issue the expected outcome is not happening is this: there is a bug in the EBT implementation of go-ssb. It won't redownload your own feed after a restore. If you connect to a peer that only supports legacy gossip then your feed will be restored correctly. I'm going to edit the title to reflect this. We are treating this as an EBF and will try to get a fix out ASAP.

@mplorentz
Copy link
Member

Another set of steps to reproduce look like this:

  1. Set up a Planetary account with some published posts
  2. Delete Planetary from the device
  3. Reinstall Planetary
  4. Open Planetary
    Expected: your feed is redownloaded.
    Actual: your feed is not downloaded.

One way to fix this is to find a peer that doesn't support EBT replication and connect to them manually. You can connect manually from the Debug menu by going to Go Bot -> paste a multi server address in the "Connect to Peer" box -> tap "Connect".

@mplorentz mplorentz changed the title "Planetary Help" auto-follow ignores forking protection Planetary will not redownload your own feed Sep 29, 2022
@mplorentz mplorentz changed the title Planetary will not redownload your own feed Planetary will not download your own feed Sep 29, 2022
@mplorentz
Copy link
Member

I found a temporary fix for this issue today. If we disable EBT replication on our pubs, and fix another bug in go-ssb (#875) so that it correctly falls back to legacy gossip then it gets feed restoration working. Unfortunately this requires a deployment of the iOS app. I am aiming to release it on Monday.

I'm also not convinced that the EBT issue is even in go-ssb, it may be in the ssb-ebt or epidemic-broadcast-trees](https://github.com/ssbc/epidemic-broadcast-trees) Node.js plugins that our pubs use. I wrote up some weird behavior I am seeing here: ssbc/ssb-ebt#59.

@mplorentz
Copy link
Member

@boreq and I confirmed on Friday that this issue is not actually in go-ssb. This is an issue in the ssb-ebt Node.js plugin used on our pubs.

@rabble
Copy link
Contributor

rabble commented Oct 10, 2022

Let's reach out to @mixmix about helping fix this because it's a bug in the JS ebt libraries.

@mplorentz
Copy link
Member

Update: Mix and I are working on a test to reproduce this.

@rabble
Copy link
Contributor

rabble commented Oct 24, 2022

@boreq and @mixmix will check in on this.

@rabble
Copy link
Contributor

rabble commented Nov 7, 2022

@mixmix what's the status of this getting fixed in the JS stack

@boreq boreq added the bug Something isn't working label Nov 21, 2022
@mplorentz
Copy link
Member

The temporary fix I described in #847 (comment) was undone (because those pubs totally stopped working with no error messages 🤷‍♂️ ) so I have opened #985 as another workaround.

@martindsq
Copy link
Member

@mplorentz should we close this? scuttlego is definitely pulling the user's feed

@mplorentz
Copy link
Member

Yeah we can close this. Scuttlego is only pulling our own feed because Filip wrote some workaround code to use legacy gossip in that case. But it does fix the bug in Planetary.

It would be nice to get rid of the workaround someday. I still think there is a bug in the javascript implementation of EBT, but I already spent over 40 hours trying to solve it myself, @mixmix spent many hours as well, and @andrestaltz says its fixed and closed the issue so I don't see what more can be done here on our end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

6 participants