Adam ⚛ Stegman

I fixed the tests on master, so you should also merge/rebase master again.

This is probably due to #128, because our gem is not setting `name_identifier_value`. When that value is missing, the format is ignored so a transient value can be generated: https://github.com/onelogin/ruby-saml/blob/eae81404d8661046aa053d0f4b4fa80f569f9f17/lib/onelogin/ruby-saml/logoutrequest.rb#L114....

This will be resolved in #149.

You're right, generally I prefer that we delegate responsibility to Devise—this is a plugin, not a full authentication system. > it would be more useful if I can specify it...

I think you're going to have to modify existing code anyway—see the "logout" section of that wiki page. During logout, your app should redirect to the IdP so the user...

@anantaUM check out the `saml_update_resource_hook` configuration: https://github.com/apokalipto/devise_saml_authenticatable/blob/37d457b394b63f360ac0f784ab35c5ceedf4e779/lib/devise_saml_authenticatable.rb#L117-L121. You can override that to add custom behavior when updating user attributes.

Thanks for your patience while I was busy and not watching issues! It's very possible that Google doesn't like a localhost URL. I've used localhost with OneLogin but not with...

The controller redirects to Google using the SAML config from `config/initializers/devise.rb`. It's a little abstracted but here's where it happens: https://github.com/apokalipto/devise_saml_authenticatable/blob/master/app/controllers/devise/saml_sessions_controller.rb#L16. Google shows a login form if one is needed,...

I might be misunderstanding what you're trying to do, but if that certificate and key are for signing requests sent to the IdP, it's [part of the settings](https://github.com/onelogin/ruby-saml#signing) you configure...

rexml is a transitive dependency of ruby-saml, so bundler should install it for you without issue. What happened in your case?