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
pouchdb-http: unmaintained #8014
Comments
db
.changes({
since: seq || 'now',
live: true
})
.on('change', change => o.next(change))
.on('error', err => o.error(err)) |
The problem is we are using That's going to be a problem going forwards... |
Ideas on fixes? Is the path going forward replacing |
Is it possible to add a deprecation warning to the package? |
module.exports = require('pouchdb-core')
.plugin(require('pouchdb-adapter-http')) and 2 dependecnies "pouchdb-adapter-http": "^6.0.3",
"pouchdb-core": "^6.0.3" so reproducing those 2 lines of code inside Your project but using PouchDB 7 would be a solution. Essentially follow instructions on this page. |
@AGrzes: Yes, we don't use this package anymore. However, other people might be using it, thinking they have the latest version of pouch not knowing this package is no longer maintained. We were very surprised why we were seeing old bugs until we realized that this package wasn't on pouch 7.x. |
@ronag You are right - and I believe @nolanlawson is both maintainer of the package github repository and collaborator on the npm package so can we please get deprecation warning |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
You can’t close a issue that’s a problem |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
When running i live changes feed using the http adaptor there seem to be a memory leak here, https://github.com/pouchdb/pouchdb/blob/master/packages/node_modules/pouchdb-adapter-http/src/index.js#L1009
Looks like the http adapter will default to
return_docs: true
even when running in "live" mode which means that it will eventually run out of memory.The text was updated successfully, but these errors were encountered: