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

Decode url-safe style base64 #10

Merged
merged 3 commits into from
Dec 31, 2014
Merged

Conversation

jessetane
Copy link
Contributor

In trying to help improve buffer's compatibility with node I came across this:

nodejs/node-v0.x-archive@b9127eb

It might be nice to have this here, but it could probably also be done in buffer, open to suggestions.

@feross
Copy link
Collaborator

feross commented Dec 18, 2014

👍

@feross
Copy link
Collaborator

feross commented Dec 18, 2014

@beatgammit Do you have any thoughts on this? I think it would be nice to have. It's backwards-compatible and unobtrusive since it only makes decode more robust, but doesn't change encode.

And it's in node's Buffer 'base64' handling too.

@beatgammit
Copy link
Owner

I'm travelling today, but I'll try to look at it this weekend.

@feross
Copy link
Collaborator

feross commented Dec 24, 2014

@beatgammit Ping!

@feross
Copy link
Collaborator

feross commented Dec 29, 2014

@beatgammit If you're too busy to take a look, are you cool if I just merge and publish a new version? I guess times like these are when it makes sense to use my commit bit, when other maintainers are too busy :)

feross added a commit that referenced this pull request Dec 31, 2014
@feross feross merged commit c92cd30 into beatgammit:master Dec 31, 2014
@feross
Copy link
Collaborator

feross commented Dec 31, 2014

Okay, I decided to go for it. Merged and released 0.0.8.

@beatgammit
Copy link
Owner

Cool, thanks! I've been travelling the last couple weeks and haven't had time.

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.

3 participants