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

Already on GitHub? Sign in to your account

allow escaping buffers in querystring.stringify and a little improvement... #5829

wants to merge 1 commit into


None yet
2 participants

zxqfox commented Jul 10, 2013

... in unescapeBuffer

Check please line 108. Do we really need toString() there?

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

Commit zxqfox/node@5c9a9a4 has the following error(s):

  • First line of commit message must be no longer than 50 characters
  • Commit message must indicate the subsystem this commit changes

The following commiters were not found in the CLA:

  • Alexej Yaroshevich

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

querystring: escape buffer objects
and a little improvement in unescapeBuffer

@zxqfox zxqfox closed this Feb 13, 2015

@zxqfox zxqfox deleted the zxqfox:querystring-stringify-buffers branch Feb 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment