jaredhanson/passport — pull request #203 by ryanmeador (2014-02-02 03:36:25+00:00)
Title: switch the arguments to the redirect function

Express expects these arguments to be the other way around. The original code throws a TypeError: Object 302 has no method 'indexOf'. My new change properly redirects (tested using passport-google-oauth 0.1.5, passport 0.2.0, express 3.0.0-rc3). Unfortunately, your unit tests no longer pass -- there are two relating to this functionality that now fail, and I'm not familiar enough with your unit test framework to figure out why they're failing. I think something in the unit test framework might also have the arguments backwards.