panva / openid-client

OAuth 2 / OpenID Connect Client API for JavaScript Runtimes
MIT License
1.83k stars 392 forks source link

update interface AuthenticateOptions to include redirect_uri #679

Closed huineng closed 4 months ago

huineng commented 4 months ago

"openid-client": "^5.6.5",

Hi , I'm referring to this discussion

https://github.com/panva/node-openid-client/pull/132#issuecomment-432792388

especially where you use

app.use('/init', (req, res, next) => {
    // figure out which redirect_uri to use and use it on the fly
    const redirect_uri = 'http://lvh.me:3000/oidc/cb';
    passport.authenticate('oidc', { redirect_uri })(req, res, next);
  });

nevertheless redirect_uri is not available on the AuthenticateOptions interface image

can it be added ?

thanks

panva commented 4 months ago

Hi @huineng

AuthenticateOptions interface is not from openid-client, it's from passport.