Upd: saml: pass callbackURL as callbackUrl. [REBASED]

[cadesalaberry]

Description

As specified in the passport-saml package, the callback URL should be passed as callbackUrl which is different from all passport strategies. To avoid such discrepancies, defaults the callbackUrl to the given callbackURL. In any case, it will not break compatibility as it will be overridden by callbackUrl if it is already specified.

callbackUrl: full callbackUrl (overrides path/protocol if supplied) path: path to callback; will be combined with protocol and server host information to construct callback url if callbackUrl is not specified (default: /saml/consume) protocol: protocol for callback; will be combined with path and server host information to construct callback url if callbackUrl is not specified (default: http://) host: host for callback; will be combined with path and protocol to construct callback url if callbackUrl is not specified (default: localhost)

Checklist

• [ ] New tests added or existing tests modified to cover all changes
• [x] Code conforms with the style guide

[slnode]

Can one of the admins verify this patch? To accept patch and trigger a build add comment ".ok\W+to\W+test."

[slnode]

Can one of the admins verify this patch?

[slnode]

Can one of the admins verify this patch?

[b-admike]

@slnode test please

[cadesalaberry]

Thanks for the merge!

Cheers,