-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Description
There are a couple of things that could be improved.
Instead of creating for every small change a issue, keep track of theme here.
-
unref()all "keep alive"/"broken connection detection" timers + cron + BroadcastChannel's - remove env var
DATABASE_UPDATE_DEBOUNCE_TIMER, context: https://stackoverflow.com/q/77847416/5781499 ¹ - improve signal handling:
npm run testruns forever/do not exit #435 (comment) ¹ -
unrefnpm run testchild process - Add config for bin pathes (npm/tar), e.g.
BIN_PATH_TAR=/usr/lib/tar,BIN_PATH_NPM=/usr/bin/npmor/usr/bin/env npm//usr/bin/env tar - Fix or at least create a list of
TODO/NOTEcomments - Remove all files prefixed with
!, e.g.!class.commands.js - Remove the
fieldsproperty fromclass.logger.js: https://github.com/OpenHausIO/backend/blob/d17066bd3bd43878371d2fa618651068c95ed78c/system/logger/class.logger.js#L50?plain=1 - Rename environment variable
HTTP_ADDRESStoHTTP_HOST² - Remove file
ISSUES.md&NOTES.md - Update Dockerfile base container image to node v20 (currently v16)
- Add path to plugins folder (default to /plugins)
- Check if errors passed to logger functions are the first argument (e.g.:
logger.warn(err, "foo bar")and notlogger.warn("foo bar", err) - Create a
constants.js: MongoDB ID/MAC regex, Symbols, Components name array, etc.
- Improve HTTP Testing (Run postman http tests from
npm run test#370 (comment))
¹) Needs further investigation if it breaks something or so...
²) Breaking change!
Metadata
Metadata
Assignees
Labels
No labels