go-kivik / pouchdb Goto Github PK

PouchDB 8.0.0 was released, and adds a few new features that should be supported by this package.

• active tasks https://pouchdb.com/api.html#active_tasks
• purge https://pouchdb.com/api.html#purge

You're doing great work, thank you!

I'm currently writing a Kivik driver for https://github.com/cayleygraph/cayley see: dennwc/cayley#4

Currently the tests all pass, but there is still work to do to make it practical to use. One of the issues I have come up against is the size of the generated JS.

To get the size of the generated JS for Cayley down to a reasonable level I had to remove the "net" dependency from the UUID library. Because "net" it is not used in the browser, it is redundant code. But using the "net" library inflates the size of the transpiled JS code by 673k bytes. You can see my PR to do this here: pborman/uuid#40

I believe the same issue exists for go-kivik/pouchdb and indeed for the main kivik repo, since it uses "net/http" which uses "net".

https://github.com/gopherjs/gopherjs/wiki/JavaScript-Tips-and-Gotchas says:

For making HTTP requests, don't use net/http, as importing this will also compile-in the entire TLS stack! Use GopherJS bindings to XmlHTTPRequest instead. For code that'll be compiled either to Go or JS, use a function that's constrained by build tags, so in JS it uses XHR and in Go it uses net/http.

I appreciate this is a pretty major change for both repos, which is why I have not simply raised a PR for this one to replace the reference to http.StatusNotFound in pouchdb.go with a local version of the constant, but rather chosen to raise this discussion issue.

To match functionality with CouchDB, ViewCleanup() returns immediately, which means that errors, including permissions errors, are never returned. Perhaps this should be changed; callers can use a goroutine if they don't want to block.