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

Broken when replicating attachments #51

Open
nolanlawson opened this issue Jun 12, 2016 · 8 comments
Open

Broken when replicating attachments #51

nolanlawson opened this issue Jun 12, 2016 · 8 comments

Comments

@nolanlawson
Copy link
Member

See pouchdb-community/pouchdb-dump-cli#14

@nolanlawson nolanlawson changed the title Broken in 5.4.0 Broken in pouchdb@5.4.0 Jun 12, 2016
@nolanlawson
Copy link
Member Author

OK, I can reproduce by pouchdb-dumping a database that contains an attachment. It only occurs when there are attachments.

@nolanlawson
Copy link
Member Author

The problem is that the writeable-stream adapter needs to define a _get now. Unfortunately due to pouchdb/pouchdb#5018 the _get method is now required for adapters of type http.

@nolanlawson nolanlawson changed the title Broken in pouchdb@5.4.0 Broken when replicating attachments Sep 12, 2016
@WorldMaker
Copy link

Is this still an issue? I was contemplating trying something wild like WebTorrent as the backend for PouchDB replication in a somewhat peer-to-peer fashion, but for my projects there are enough attachments (photos) that it doesn't make sense to try to build something like that if it won't support attachments.

@nolanlawson
Copy link
Member Author

Yes I haven't had time to fix it. PRs welcome. :)

@WorldMaker
Copy link

I'm hitting some issues where I may need a replicable export dump in the near future, and as mentioned attachment support is important to the projects I'm working on. I'm trying to gauge if it would be worth trying to integrate that into this project or start "fresh" with a file format that more easily supports large binary attachments as "first-class" than newline-delimited JSON (such as using one of the JS-based zip file libraries).

@nolanlawson
Copy link
Member Author

The problem isn't JSON vs non-JSON, it's that pouchdb-replication-stream needs to implement the _get method.

@WorldMaker
Copy link

In my use-case of many, potentially large image files, base64 overhead is possibly a concern, that's what lead to me considering maybe a zip-based format. Especially too because that makes it easier for users to extract the zip to explore things like embedded photos as opposed to needing a base64 decoder.

@rileyai-dev
Copy link

@nolanlawson I'd be happy to help if you give me a little guidance... Was it solved in the cli?

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