remove js polyfill for Array.from - Githubissues

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

57 stars 9 forks source link

remove js polyfill for Array.from #17

Closed ivanitskiy closed 1 year ago

ivanitskiy commented 1 year ago

Describe the bug

NJS version 0.8.0 added native implementation for Array.from: https://mailman.nginx.org/pipermail/nginx-announce/2023/A3O7JLCSI6DYGIYQA4XCXLJEUW2EGIHS.html

     *) Feature: added Array.from(), Array.prototype.toSorted(),
         Array.prototype.toSpliced(), Array.prototype.toReversed().

Now we can remove: https://github.com/nginxinc/njs-acme/blob/274755f50c2ae86a6fed4fd17b359e6f30d02faa/src/arrayFrom.js#L1

Your environment

Version/release of this project or specific commit
Target deployment platform

Additional context

Add any other context about the problem here.

ivanitskiy commented 1 year ago

resolved in https://github.com/nginxinc/njs-acme/pull/21/