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

Regression: Sync failure after delete record (when using quickstart) (v 0.7.5) #102

Closed
michaelneale opened this issue May 31, 2023 · 2 comments · Fixed by #103
Closed

Regression: Sync failure after delete record (when using quickstart) (v 0.7.5) #102

michaelneale opened this issue May 31, 2023 · 2 comments · Fixed by #103
Labels
bug Something isn't working

Comments

@michaelneale
Copy link
Contributor

michaelneale commented May 31, 2023

Following the quickstart with 0.7.5 web5: https://developer.tbd.website/docs/web5/quickstart

code:
https://gist.github.com/michaelneale/282ea256fc936cca2cc9bfd0087d6903

....
Results in an error:

secp256k1 unavailable, reverting to browser version
Sync failed due to error:  Error: (404) Failed to read data associated with record bafyreig3knfootf2o7f5tladyfkvcc3w2wojp2hwa7uwn6hrj7fexfahzy. Not Found}
    at _SyncApi.<anonymous> (file:///Users/micn/Documents/code/web5-app/node_modules/@tbd54566975/web5-user-agent/dist/esm/main.mjs:977:17)
    at Generator.next (<anonymous>)
    at fulfilled (file:///Users/micn/Documents/code/web5-app/node_modules/@tbd54566975/web5-user-agent/dist/esm/main.mjs:46:24)

Update: it does delete it - but then sync kills it:

delete result { status: { code: 202, detail: 'Accepted' } }
Sync failed due to error:  Error: (404) Failed to read data associated with record bafyreib56qrdl2o4b3mhrlqha7pj767xgttzhrspcn4faetb63pwtfybqu. Not Found}
    at _SyncApi.<anonymous> (file:///Users/micn/Documents/code/web5-app/node_modules/@tbd54566975/web5-user-agent/dist/esm/main.mjs:977:17)
    at Generator.next (<anonymous>)
    at fulfilled (file:///Users/micn/Documents/code/web5-app/node_modules/@tbd54566975/web5-user-agent/dist/esm/main.mjs:46:24
@michaelneale
Copy link
Contributor Author

@mistermoe you can see it after it tries to sync

@michaelneale michaelneale changed the title 404 on sync (when using quickstart) (v 0.7.5) Unable to delete record: 404 on sync (when using quickstart) (v 0.7.5) May 31, 2023
@michaelneale michaelneale changed the title Unable to delete record: 404 on sync (when using quickstart) (v 0.7.5) Regression: Unable to delete record: 404 on sync (when using quickstart) (v 0.7.5) May 31, 2023
@michaelneale michaelneale added the bug Something isn't working label May 31, 2023
@michaelneale michaelneale changed the title Regression: Unable to delete record: 404 on sync (when using quickstart) (v 0.7.5) Regression: Sync failure after delete record (when using quickstart) (v 0.7.5) May 31, 2023
@mistermoe mistermoe linked a pull request Jun 1, 2023 that will close this issue
@mistermoe
Copy link
Contributor

@michaelneale thanks for the reproducible gist

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
None yet
Development

Successfully merging a pull request may close this issue.

2 participants