[BUG] Unable to override default authnRequestBinding when using MultiSamlStrategy

[mhesler74]

passport.authenticate only uses the default authnRequestBinding setting when using the MultiSamlStrategy class.
Providing an alternative authnRequestBinding setting as a return value in the getSamlOptions function is completely ignored.

[cjbarth]

Would you be willing to make a PR to address this @mhesler74 ?

[stavros-wb]

It's currently a "global" passport configuration

passport-saml/test/multiSamlStrategy.js

Line 136 in 43465d6

authnRequestBinding: 'HTTP-POST',

Do you need it, to be set per client?

[mhesler74]

Yes, I need it set per client. Some of our customers are using HTTP-Request binding for authentication while others support HTTP-POST.

[mhesler74]

Actually another PR was put in a while back by someone else... very similar
#432

[mhesler74]

Any ETA on when this will be included in a release?

[cjbarth]

I believe this is part of the v3 release, which is held up due to some missing code reviews. See https://github.com/node-saml/passport-saml/milestone/1.