Tweak docker build
Ignore the full set of files that should be ignored, make the container build with a UID/GID so it can run with the same perms as nginx on the host and share a unix socket.
(Build args are passed like so: docker build --build-arg="UID=$(id nginx -u)" --build-arg="GID=$(id nginx -g)" -t "$IMAGE_NAME" -f "./.dockerfile" .)
Fix an error in conversion from forEach to for
Early return from forEach is return, but in a previous commit this was converted to for of, where it should be continue (good spot @SkyeOfBreeze)
I also made this a plain for loop with var, since it's a hot path. Microbenchmarks have lots of problems, but since I'm trying to see if this specific code path can improve CPU usage, I'll take what bits I can get. The loop portion of the code is not where the cpu usage is going, but eh. If my other code goes well, this'll be all deferred to C++ with much better pubsub performance anyway.
(Build args are passed like so:
docker build --build-arg="UID=$(id nginx -u)" --build-arg="GID=$(id nginx -g)" -t "$IMAGE_NAME" -f "./.dockerfile" .
)forEach
tofor
Early return fromforEach
isreturn
, but in a previous commit this was converted tofor of
, where it should becontinue
(good spot @SkyeOfBreeze)I also made this a plain for loop with var, since it's a hot path. Microbenchmarks have lots of problems, but since I'm trying to see if this specific code path can improve CPU usage, I'll take what bits I can get. The loop portion of the code is not where the cpu usage is going, but eh. If my other code goes well, this'll be all deferred to C++ with much better pubsub performance anyway.