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
190 stars 175 forks source link

ReferenceError: ex is not defined #1

Open shekhei opened 11 years ago

shekhei commented 11 years ago

ex is not defined at lib/strategy.js:170

        } catch(e) {
          return self.error(ex);
        }
flystax commented 11 years ago

from their own code:

              } catch(ex) {
                return self.error(ex);
              }

the author missed an x here.

shekhei commented 11 years ago

yup that's exactly the problem

warrenseine commented 8 years ago

Hi @jaredhanson,

Thanks for making this strategy. It works fine even it seems to be abandoned. However, the npm package 0.0.1 seems to be even more outdated, as this is issue is still in the npm package, but not on the repository.

Even if there's no more contribution, could you push a 0.0.2 with the current GitHub version?

fiddur commented 7 years ago

This is solved; all catch uses ex now.