tinyhttp / tinyhttp / 10042518769

98%

push

github

Feat/trust proxy (#430)

* refactor: appropriately allow & handle heterogeneous arrays

* refactor: replace explicit array population with `.map`

* refactor: add `Subnet` type and apply it to avoid unsafe casting

* chore: fix JSDoc parameter references

* build: enable `strict` type-checking for proxy-addr

* fix: comply with strict type-checking

* feat: implement 'hops' number as `trust` parameter

* refactor: add IPv4 and IPv6 type-guards to avoid type-casts

* refactor: annotate `trust` parameter as optional

* feat: add 'trust proxy' setting

* feat: default `trust proxy` setting to `0` (hops)

* feat: pass `trust proxy` app setting to network extensions

* perf: pre-compile `trust proxy` setting to avoid repeated computation

* test(proxy-addr): nest `trust` test group inside `arguments` group

* test(proxy-addr): add `arguments.req` test group

* test(proxy-addr): fix type error related to signature of `TrustFunction`

* test(proxy-addr): add `argument.trust` `should be required` test

* test(proxy-addr): rename duplicate test

* test(proxy-addr): add `arguments.trust` `it should not alter input array` test

* test(proxy-addr): rename duplicate test

* test(proxy-addr): add no-op tests to match cases from jshttp/proxy-addr

* test: rewrite `createReq` helper to avoid `as IncomingMessage` cast every time it is used

* test(proxy-addr): remove `as IncomingMessage`

* test(proxy-addr): tidy-up header block

remove unused import `IncomingMessage`
import `all`, `compile` from `proxy-addr`
rename `all` -> `trustAll`
rename `none` -> `trustNone`
refactor `function trust10x` to `const trust10x`

* test(proxy-addr): implement no-op testcases

* test(proxy-addr): rewrite 'should reject' tests

* fix(proxy-addr): `.slice()` input array to prevent mutating it

* fix(proxy-addr): adjust behaviour to match jshttp/proxy-addr

* chore: bump `app`, `proxy-addr` and `jsonp`

* test(app): add tests... (continued)

761 of 806 branches covered (94.42%)

Branch coverage included in aggregate %.

101 of 103 new or added lines in 5 files covered. (98.06%)

3237 of 3355 relevant lines covered (96.48%)

22.84 hits per line