airbnb/hypernova

Document `onServer`?

Open

#73 geöffnet am 20. Juni 2017

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (1 zugewiesene Person)JavaScript (249 Forks)batch import
documentationhelp wanted

Repository-Metriken

Stars
 (5.831 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

I'm using onServer to be able to hook into express before the batch route is added, so I can do something like this: https://stackoverflow.com/a/21858212/4396258

(Context: we want to just restart workers if they consume too much memory, but specifically at the end of a request so we don't drop drop any requests)

It's either this or adding a workerShouldExit method as a config? What do you think? Can I safely use the onServer method, or is this subject to removal?

(Also on the subject of undocumented arguments - what about documenting context too? It's useful for collecting stats.)

@ljharb @goatslacker

Contributor Guide