Uncatchable exception when parsing entryPoint

[gunzip]

In a MultiStrategy, getSamlOptions may not find a suitable entrypoint to pass to the saml client
so we end up with entryPoint = undefined. This causes the NodeJS process to crash since these lines here: https://github.com/bergie/passport-saml/blob/master/lib/passport-saml/saml.js#L307
contain an uncheck call to url.parse which throws since entryPoint is not defined. As we are in a callback there's no way to catch the exception and the process exits. I guess the whole method should be wrapped in a try / catch block (not that this may be used as a DoS against the service provider)

[markstos]

@gunzip Are you willing to contribute a pull request to improve this? Thanks for the report.

[googamanga]

Hitting this issue to, getting unhandledPromise error if running "return done(new Error('saml not found'))" inside of getSamlOptions

[googamanga]

@gunzip is there a work around?

[cjbarth]

This problem is no longer in this repository. The affected code is now in node-saml. If the problem persists, please re-open this over there.