response_type hard coded

[mattdot]

I'm implementing a passport provider for visual studio online and they use response_type=Assertion (docs @ https://www.visualstudio.com/en-us/docs/integrate/get-started/auth/oauth ). Unfortunately passport-oauth2 hardcodes response_type=code. Would you be open to a pull request that allows derived strategies to override the response_type? I got my passport provider working by just copying the entire authorize implementation, but it seems like I shouldn't have had to do that.

https://github.com/mattdot/passport-visualstudio/blob/master/lib/strategy.js

[josefwijaya]

+1

[ceceliastory]

+1

[jeremy-ww]

+1

[brunosantanarj]

+1

[shushu304]

+1
if you can update the strategy.js line 217 to :
params.response_type = params.response_type || 'code';
it will be great.

[robballou]

Related PR #79