feross/buffer

Support 'buffer' encoding

Open

#227 geöffnet am 19. Feb. 2019

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (259 Forks)github user discovery
enhancementhelp wanted

Repository-Metriken

Stars
 (1.883 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Since Node.js v10.0.0 crypto.DEFAULT_ENCODING is deprecated and the default value is now 'buffer', which makes methods default to Buffer objects. I am running into an issue where Buffer.from is called on a string, but the encoding is buffer. That throws an exception because Buffer.isEncoding doesn't support the buffer encoding.

I'd suggest to add this to allow users to integrate this with a more up-to-date version of node.

Would it already work if we just adjust Buffer.isEncoding and add buffer as one of the supported types? Or do we need to do more to make this work?

Contributor Guide