Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Buffer.from instead of new Buffer #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TehShrike
Copy link

The buffer constructor has been deprecated since node 6, which fell out of maintenance a good while ago.

atob-lite was similarly updated a couple years ago: https://github.com/hughsk/atob-lite/blob/master/atob-node.js

The buffer constructor has been deprecated since node 6.
@gr2m
Copy link

gr2m commented Sep 1, 2019

ping ☝️

@Richienb
Copy link

@hughsk ☝️

@Richienb
Copy link

Richienb commented Aug 7, 2020

@hughsk

@gr2m
Copy link

gr2m commented Aug 7, 2020

@Richienb maybe we should maintain our own btoa polyfill lib within @node-fetch

@Richienb
Copy link

Richienb commented Aug 8, 2020

@gr2m I just created https://github.com/node-fetch/btoa-lite. Any idea who owns the node-fetch npm org?

@gr2m
Copy link

gr2m commented Aug 8, 2020

I assume @bitinn

@Richienb
Copy link

Richienb commented Aug 8, 2020

// @bitinn

@Richienb
Copy link

@bitinn

@bitinn
Copy link

bitinn commented Apr 21, 2022 via email

@Richienb
Copy link

@gr2m suggested to publish the module as @node-fetch/btoa-lite and assumes that you are the person with the publish perms.

@bitinn
Copy link

bitinn commented Apr 21, 2022

@gr2m suggested to publish the module as @node-fetch/btoa-lite and assumes that you are the person with the publish perms.

I think we never had a npm org, we publish module individually (see node-fetch and fetch-blob). Jimmy owns publishing right for both of them now. For btoa-lite, you can certainly help to setup a npm org :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants