jaredhanson / passport-openid

OpenID authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-openid/?utm_source=github&utm_medium=referral&utm_campaign=passport-openid&utm_content=about
MIT License
98 stars 85 forks source link

Avoid 'undefined undefined' displayName #7

Closed lortabac closed 11 years ago

lortabac commented 11 years ago

When a provider does not provide the full name of the user, displayName is set to 'undefined undefined', which is annoying.

Added the condition to only set displayName if firstname and lastname parameters are set, so applications can handle empty names.

jaredhanson commented 11 years ago

Thanks! Merged and published to npm as passport-openid v0.3.0.