Permalink
Browse files

encodeURIComponent is redundant

`proof` is `hex` encoded, and is URI safe. Remove redundant URI encoding attempt.
  • Loading branch information...
1 parent 5387f47 commit 732121dd8ea09e240e24e4fc9e2e6cd0d428b758 @suryagh suryagh committed May 16, 2016
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/strategy.js
View
@@ -143,7 +143,7 @@ Strategy.prototype.userProfile = function(accessToken, done) {
// For further details, refer to:
// https://developers.facebook.com/docs/reference/api/securing-graph-api/
var proof = crypto.createHmac('sha256', this._clientSecret).update(accessToken).digest('hex');
- url.search = (url.search ? url.search + '&' : '') + 'appsecret_proof=' + encodeURIComponent(proof);
+ url.search = (url.search ? url.search + '&' : '') + 'appsecret_proof=' + proof;
}
if (this._profileFields) {
var fields = this._convertProfileFields(this._profileFields);

0 comments on commit 732121d

Please sign in to comment.