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

Add timeout for find queries #40

Open
wants to merge 72 commits into
base: master
Choose a base branch
from
Open

Conversation

SirAnthony
Copy link

Browser usually drops connection after 5 minutes as timeout request, but find query continues to execute and create cpu load. This behaviour can be reproduced with regexp query on big dbs (millions of documents).

alexb and others added 30 commits September 4, 2020 21:17
fix removing from document's page
* feat: read only mode (huggingface#72)

* feat: read only mode

* bumped most dependencies' versions

* feat(server): update deps and little cleanup

* Cleanup & revert a few things

* fix(server): change middleware 401 response to empty json

* Cleanup a bit more

* Disable edit/remove buttons on readOnly mode

* Fix error when attempting to edit in readOnly mode

* Fix Dockerfile for readOnly

* Actually fix readOnly propagation to client

* document read-only mode in readme

* Simplifiy + predictible dockerfile

* refacto: reply with 405 in RO mode

* refacto: rename `readOnlyMode` middleware

Co-authored-by: Anthony MOI <[email protected]>
Co-authored-by: Pierric Cistac <[email protected]>
Co-authored-by: Adrien <[email protected]>
Co-authored-by: Eliott C. <[email protected]>

* feat(mongo): password less support

* Revert changes reverted by merge

* Remove unneeded file

* Remove unneeded file

* Do not add package-lock to the tree

* Fix compilation

* Fix compilation

* Fix build

* Default value for query

---------

Co-authored-by: Luc Georges <[email protected]>
Co-authored-by: Anthony MOI <[email protected]>
Co-authored-by: Pierric Cistac <[email protected]>
Co-authored-by: Adrien <[email protected]>
Co-authored-by: Eliott C. <[email protected]>
Co-authored-by: rtrompier <[email protected]>
Co-authored-by: Remy <[email protected]>
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.

5 participants