search

jaredhanson / passport-openidconnect

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

Bug Fix: 'openid' added to scope even if already present #99

Open mstrhakr opened 1 year ago

mstrhakr commented 1 year ago

This plugin adds the 'openid' scope to the request, even if that scope is already included, this can cause problems with some IdPs. (closes #98)

This pull fixes the problem by adding a check while building the params to see if 'openid' is already present in the scope, adding it if necessary.

This is tested and working with a combination of Meshcentral and Authelia