cmorten / opine

Minimalist web framework for Deno ported from ExpressJS.

https://github.com/cmorten/opine/blob/main/.github/API/api.md

MIT License

854 stars 43 forks source link

[NO-ISSUE] Support Deno 1.9.2 #118

Closed cmorten closed 3 years ago

cmorten commented 3 years ago

Issue

No issue

Details

Support Deno 1.9.2.

Note some adjustments to the behaviour of res.set() and res.append() due to the Headers class changing in Deno ^1.9.0 to be specification compliant. Given it was previously non-compliant, these changes are being viewed as bug fixes rather than breaking changes.

Trace is also now considered a forbidden method by Deno.

CheckList

[ ] PR starts with [#_ISSUEID].
[ ] Has been tested (where required).

github-actions[bot] commented 3 years ago

Benchmark results

PR to merge feat/deno-1.9.2 dd58f2a2fc3e5cbe8d3c2308e157d25b61848f8c -> main

``` 1 middleware 7672.05 RPS 5 middleware 6890.68 RPS 10 middleware 7310.01 RPS 15 middleware 7006.03 RPS 20 middleware 6149.79 RPS 30 middleware 6363.95 RPS 50 middleware 5667.84 RPS 100 middleware 3930.49 RPS ```