passport dependency with passport-saml dependency

krakenjs / passport-saml-encrypted

A strategy for Passport authentication that supports encrypted SAML responses

MIT License

14 stars 26 forks source link

passport dependency with passport-saml dependency #10

Open jketo opened 9 years ago

jketo commented 9 years ago

Trying to use passport 0.3.0

Getting error: TypeError: Cannot set property 'user' of undefined [TypeError: Cannot set property 'user' of undefined] TypeError: Cannot set property 'user' of undefined at /project/node_modules/passport-saml-encrypted/node_modules/passport/lib/http/request.js:50:35

Seem's to be need same fix as fixed on https://github.com/bergie/passport-saml/commit/010f2d4373719cf34b6dc241d51e76d8e4b1c2f2

lmarkus commented 9 years ago

I'm up to my neck right now, and can't really verify. Would you mind checking if this works on this repo as well? (If it does, and you send me a PR, I'd be very grateful! )

jketo commented 9 years ago

Ok. Done and tested on openidp.feide.no