profile and cb are flipped in callback function when defining strategy to use

[sbernheim4]

In regards to setting up a strategy, specifically the callback like so:

const Strategy = require('passport-facebook').Strategy;
passport.use(new Strategy({
    clientID: ****,
    clientSecret: ****,
    callbackURL: ****
},
    function(accessToken, refreshToken, profile, cb) {

        });
)

I believe in this example, the value of profile and cb are swapped. In my app running console.log(profile) prints out undefined whereas console.log(cb) prints out the actual profile information. For example:

{ id: '****',
  username: undefined,
  displayName: 'Sam Bernheim',
  name:
   { familyName: undefined,
     givenName: undefined,
     middleName: undefined },
  gender: undefined,
  profileUrl: undefined,
  provider: 'facebook',
  _raw: '{"name":"Sam Bernheim","id":"****"}',
  _json: { name: 'Sam Bernheim', id: '****' } }

This makes sense since I never defined cb. To me this means that the values just need to be flipped and the function should look like function(accessToken, refreshToken, cb, profile) or better yet the implementation should be fixed to keep the order consistent with the other strategies (assuming they work and don't have this problem as well)