7 Commits (main)

Author	SHA1	Message	Date
Florent	4a9b6fea9d	Shutdown Doc worker when it is not considered as available in Redis #831 (#856) ... * Shutdown Doc worker when it is not considered as available in Redis * Use isAffirmative for GRIST_MANAGED_WORKERS * Upgrade Sinon for the tests * Run Smoke test with pages in English * Add logic in /status endpoint	2 months ago
Dmitry S	fbae81648c	(core) Add options to /status health-check endpoints to check DB and Redis liveness. ... Summary: - /status accepts new optional query parameters: db=1, redis=1, and timeout=<ms> (defaults to 10_000). - These verify that the server can make trivial calls to DB/Redis, and that they return within the timeout. - New HealthCheck tests simulates DB and Redis problems. - Added resilience to Redis reconnects (helped by a test case that simulates disconnects) - When closing Redis-based session store, disconnect from Redis (to avoid hanging tests) Some associated test reorg: - Move stripeTools out of test/nbrowser, and remove an unnecessary dependency, to avoid starting up browser for gen-server tests. - Move TcpForwarder to its own file, to use in the new test. Test Plan: Added a new HealthCheck test that simulates DB and Redis problems. Reviewers: georgegevoian Reviewed By: georgegevoian Differential Revision: https://phab.getgrist.com/D4054	8 months ago
Alex Hall	0005ad013e	(core) Notify open ActiveDocs when the product is upgraded ... Summary: When an account is upgraded to a new product in Billing, send a message to the redis channel `billingAccount-${accountId}-product-changed`. ActiveDocs subscribe to this channel. When a message is received, they refresh their product from the database and use it to recalculate doc usage based on new limits. The new usage is broadcast to clients so they see the result of the upgrade live. Test Plan: Extended nbrowser Billing test to test that a document open in a separate tab has its limit banner cleared immediately on upgrade. Reviewers: paulfitz Reviewed By: paulfitz Differential Revision: https://phab.getgrist.com/D3480	2 years ago
Alex Hall	0beb2898cb	(core) Add flexibility to daily API usage limit ... Summary: Allow exceeding the daily API usage limit for a doc based on additional allocations for the current hour and minute. See the doc comment on getDocApiUsageKeysToIncr for details. This means that up to 5 redis keys may be relevant at a time for a single document. Test Plan: Updated and expanded 'Daily API Limit' tests. Reviewers: dsagal Reviewed By: dsagal Differential Revision: https://phab.getgrist.com/D3368	2 years ago
Alex Hall	2c9ae6dc94	(core) Enforce daily limit on API usage ... Summary: Keep track of the number of API requests made for this document today in redis. Uses local caches of the count and the document so that usually requests can proceed without waiting for redis or the database. Moved the free standing function apiThrottle to become a method to avoid adding another layer of request handler callbacks. Test Plan: Added a DocApi test Reviewers: paulfitz Reviewed By: paulfitz Subscribers: dsagal Differential Revision: https://phab.getgrist.com/D3327	2 years ago
Alex Hall	4a70283292	(core) Webhook event queue on redis ... Summary: Push webhook events to redis queue with key based on docId. Remove events from redis after sending using LTRIM. Put failed events back on the end of the queue under normal circumstances. When the event queue gets too long: - Wait until it gets consumed before continuing. - Drop failed events (i.e. don't put them back on the end of the queue) - Limit webhook retries to 5 Test Plan: Tested that interactions with redis are as expected using redis MONITOR command. Reviewers: paulfitz Reviewed By: paulfitz Differential Revision: https://phab.getgrist.com/D3100	3 years ago
Paul Fitzpatrick	5ef889addd	(core) move home server into core ... Summary: This moves enough server material into core to run a home server. The data engine is not yet incorporated (though in manual testing it works when ported). Test Plan: existing tests pass Reviewers: dsagal Reviewed By: dsagal Differential Revision: https://phab.getgrist.com/D2552	4 years ago