Merge pull request #216 from alvinward/master

Add support for ProviderName attribute

.travis.yml

@@ -6,9 +6,6 @@ node_js:
   - "4.0"
   - "stable"
 
-before_install:
-  - npm install -g npm
-
 script:
   - npm test
   - ./node_modules/.bin/jshint lib

README.md

@@ -65,6 +65,7 @@ passport.use(new SamlStrategy(
   * `disableRequestedAuthnContext`: if truthy, do not request a specific auth context
   * `authnContext`: if truthy, name identifier format to request auth context (default: `urn:oasis:names:tc:SAML:2.0:ac:classes:PasswordProtectedTransport`)
   * `forceAuthn`: if set to true, the initial SAML request from the service provider specifies that the IdP should force re-authentication of the user, even if they possess a valid session.
+  * `providerName`: optional human-readable name of the requester for use by the presenter's user agent or the identity provider
   * `skipRequestCompression`: if set to true, the SAML request from the service provider won't be compressed.
   * `authnRequestBinding`: if set to `HTTP-POST`, will request authentication from IDP via HTTP POST binding, otherwise defaults to HTTP Redirect
  * **InResponseTo Validation**
@@ -78,6 +79,7 @@ passport.use(new SamlStrategy(
   * `additionalLogoutParams`: dictionary of additional query params to add to 'logout' requests
   * `logoutCallbackUrl`: The value with which to populate the `Location` attribute in the `SingleLogoutService` elements in the generated service provider metadata.
 
+
 ### Provide the authentication callback
 
 You need to provide a route corresponding to the `path` configuration parameter given to the strategy:

lib/passport-saml/saml.js

@@ -199,6 +199,10 @@ SAML.prototype.generateAuthorizeRequest = function (req, isPassive, callback) {
       request['samlp:AuthnRequest']['@AttributeConsumingServiceIndex'] = self.options.attributeConsumingServiceIndex;
     }
 
+    if (self.options.providerName) {
+      request['samlp:AuthnRequest']['@ProviderName'] = self.options.providerName;
+    }
+
     callback(null, xmlbuilder.create(request).end());
   })
   .fail(function(err){