Dev environment + Typescript + Variables Improvements

nginx / njs-acme

Nginx NJS module runtime to work with ACME providers like Let's Encrypt for automated no-reload TLS certificate issue/renewal.

Apache License 2.0

60 stars 9 forks source link

Dev environment + Typescript + Variables Improvements #5

Closed zsteinkamp closed 1 year ago

zsteinkamp commented 1 year ago

Ergonomics for dev environment (added node container to docker-compose along with VSCode devcontainer support)
Hacking away at Typescript conformance
Added prettier for automatic code formatting
Utility methods targeting variables, with the intent of allowing the use of Nginx or environment variables for all configurable things.

ivanitskiy commented 1 year ago

LGTM with a few minor comments.

ivanitskiy commented 1 year ago

I see a missing part here, an updated examples/nginx.conf file, which documents all newly introduced (request)variables and associated env var and their respective defaults.