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

NIP-96: File Distribution over Nostr #358

Closed
wants to merge 0 commits into from

Conversation

KaffinPX
Copy link

I'm proposing and developing a solution for managing files/repositories over Nostr without storing actual data on it but on an alternate network.

I will release our CLI client tomorrow(Currently im testing&improving code quality) and our web client coming in some days(Its a working prototype rn).

Will release screenshots if comments ;)

@KaffinPX
Copy link
Author

We released our preview of web client on https://fostr.dev ;) CLI delayed to tomorrow.

@KaffinPX
Copy link
Author

Source code of web client will be released soon over same account w CLI client.

@KaffinPX
Copy link
Author

KaffinPX commented Mar 14, 2023

Also we know theres a bug on address linking over website on web client, will be fixed soon

Edit: Fixed

@akhavr
Copy link

akhavr commented Mar 14, 2023

We released our preview of web client on https://fostr.dev ;) CLI delayed to tomorrow.

Where does it store the files, actually?

@KaffinPX
Copy link
Author

We released our preview of web client on https://fostr.dev ;) CLI delayed to tomorrow.

Where does it store the files, actually?

Our implementations are using IPFS

@KaffinPX
Copy link
Author

@akhavr
Copy link

akhavr commented Mar 15, 2023

Our NIP is on https://fostr.dev/#/npub1hau0ws8e4405g5ckst669hn4znlzhh7j7tx8tqr8m3fu4hzmetxqe6r0er/fostr-protocol?path=/PROTOCOL.md now

Failed to load repository

This repository does not exist

:)

@KaffinPX
Copy link
Author

Our NIP is on https://fostr.dev/#/npub1hau0ws8e4405g5ckst669hn4znlzhh7j7tx8tqr8m3fu4hzmetxqe6r0er/fostr-protocol?path=/PROTOCOL.md now

Failed to load repository

This repository does not exist

:)

Check your internet connection :)

@KaffinPX
Copy link
Author

So yeah i can reproduce that by having a 700-800 ms ping internet(aka satellite internet), looks like its because request timeout so we will add a refresh button in next update.

@JeanOUINA
Copy link

JeanOUINA commented Mar 15, 2023

Our NIP is on https://fostr.dev/#/npub1hau0ws8e4405g5ckst669hn4znlzhh7j7tx8tqr8m3fu4hzmetxqe6r0er/fostr-protocol?path=/PROTOCOL.md now

Failed to load repository

This repository does not exist

:)

this should be linked to your internet connection. Alternatively, we just added a Refresh button to retry loading the page if needed (it is possible that not every relay is connected yet when loading the repository).

@akhavr
Copy link

akhavr commented Mar 16, 2023

this should be linked to your internet connection. Alternatively, we just added a Refresh button to retry loading the page if needed (it is possible that not every relay is connected yet when loading the repository).

Yeah, it was Starlink in a shitty location. Yet, giving such message is a bad UX.

Still trying to try. Now getting

Failed to load repository

(intermediate value).Objects is undefined
invalid path "/PROTOCOL.md": path does not begin with '/'

@KaffinPX
Copy link
Author

Uhm i cant reproduce that, can you add me on Discord: KaffinPX#0001 so we can try to get some debug data.

@frbitten
Copy link
Contributor

Is it not possible to take advantage of the [NIP-94] (#337) that already proposes this file broadcast instead of creating a new NIP?
You could just define the extra tags and content format to support your needs.

@KaffinPX
Copy link
Author

Our solution is immutable and designed for distributing decentralized file system ids, not a beacon for urls.

@akhavr
Copy link

akhavr commented Mar 19, 2023

KaffinPX#0001

Sorry, not a discord user so can't find you there. Still interested to understand how your solution works.

@KaffinPX
Copy link
Author

KaffinPX commented Mar 21, 2023

KaffinPX#0001

Sorry, not a discord user so can't find you there. Still interested to understand how your solution works.

Im going to release CLI client soon, also can you share dev console logs to here so maybe we can find smth.

@akhavr
Copy link

akhavr commented Mar 21, 2023

can you share dev console logs to here so maybe we can find smth.

It gets 500 on https://node-ipfs.thomiz.dev/5001/api/v0/cat?arg=/PROTOCOL.md:

POST | https://node-ipfs.thomiz.dev/5001/api/v0/cat?arg=/PROTOCOL.md

@KaffinPX
Copy link
Author

Thomiz will check soon

@rafael-xmr
Copy link

Failed to load repository

(intermediate value).Objects is undefined
invalid path "/PROTOCOL.md": path does not begin with '/'

This happens in firefox here, but not in chrome/brave

@KaffinPX
Copy link
Author

Failed to load repository

(intermediate value).Objects is undefined
invalid path "/PROTOCOL.md": path does not begin with '/'

This happens in firefox here, but not in chrome/brave

Ah so give us some time to fix, ig some Path standard differences

@KaffinPX
Copy link
Author

Fixed

@KaffinPX
Copy link
Author

KaffinPX commented Apr 1, 2023

Can everyone help us to advertise Fostr, literally no one knows about it yet lol

@KaffinPX
Copy link
Author

KaffinPX commented Apr 1, 2023

@fiatjaf fyi, its ready to be reviewed

@xeruf
Copy link

xeruf commented Dec 26, 2023

what happened to this, why is there another 96?
#547

@KaffinPX
Copy link
Author

what happened to this, why is there another 96? #547

This repo is centralized and this pull request got ghosted.

@fiatjaf
Copy link
Member

fiatjaf commented May 20, 2024

If this repo was centralized it would have been my responsibility to read this and decide upon merging it or not. Since it is not, then I can safely ignore it until the community is on the streets claiming for the merge.

@fiatjaf
Copy link
Member

fiatjaf commented May 20, 2024

But I personally didn't like it.

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

Successfully merging this pull request may close these issues.

7 participants