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

doc: document base64url encoding support #5243

Closed
wants to merge 1 commit into from
Closed

doc: document base64url encoding support #5243

wants to merge 1 commit into from

Conversation

tristanls
Copy link
Contributor

Documents that creating a buffer from base64 encoded string transparently
supports "URL and Filename Safe Alphabet" base64url encoding.

@tristanls
Copy link
Contributor Author

Documenting per @bnoordhuis 's suggestion in #5239

@mscdex mscdex added buffer Issues and PRs related to the buffer subsystem. doc Issues and PRs related to the documentations. labels Feb 15, 2016
@bnoordhuis
Copy link
Member

LGTM but @nodejs/documentation, for when you land this, I think there is a long line in the commit log.

@tristanls
Copy link
Contributor Author

Yes, missed it by one :(

> "Documents that creating a buffer from base64 encoded string transparently".length
73

@benjamingr
Copy link
Member

LGTM (other than the commit line thing :P)

Documents that creating a buffer from base64 encoded string
transparently supports "URL and Filename Safe Alphabet" base64url
encoding.
@tristanls
Copy link
Contributor Author

force pushed fixing line length

@a0viedo
Copy link
Member

a0viedo commented Feb 26, 2016

LGTM

silverwind pushed a commit that referenced this pull request Feb 27, 2016
Documents that creating a buffer from base64 encoded string
transparently supports "URL and Filename Safe Alphabet" base64url
encoding.

PR-URL: #5243
Reviewed-By: Ben Noorhduis <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Alejandro Oviedo <[email protected]>
@silverwind
Copy link
Contributor

Thanks! Landed in 1411e0b.

@silverwind silverwind closed this Feb 27, 2016
rvagg pushed a commit that referenced this pull request Feb 28, 2016
Documents that creating a buffer from base64 encoded string
transparently supports "URL and Filename Safe Alphabet" base64url
encoding.

PR-URL: #5243
Reviewed-By: Ben Noorhduis <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Alejandro Oviedo <[email protected]>
@Fishrock123 Fishrock123 mentioned this pull request Mar 1, 2016
5 tasks
MylesBorins pushed a commit that referenced this pull request Mar 17, 2016
Documents that creating a buffer from base64 encoded string
transparently supports "URL and Filename Safe Alphabet" base64url
encoding.

PR-URL: #5243
Reviewed-By: Ben Noorhduis <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Alejandro Oviedo <[email protected]>
MylesBorins pushed a commit that referenced this pull request Mar 21, 2016
Documents that creating a buffer from base64 encoded string
transparently supports "URL and Filename Safe Alphabet" base64url
encoding.

PR-URL: #5243
Reviewed-By: Ben Noorhduis <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Alejandro Oviedo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
buffer Issues and PRs related to the buffer subsystem. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants