{"issue":{"pull_request":{"diff_url":null,"html_url":null,"patch_url":null},"state":"closed","milestone":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/4f6fbd05e123ff2bb1d34de48424ddc1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"4f6fbd05e123ff2bb1d34de48424ddc1","url":"https://api.github.com/users/ivanjovanovic","login":"ivanjovanovic","id":23173},"assignee":null,"updated_at":"2011-11-16T11:39:03Z","labels":[],"closed_at":"2011-11-16T11:39:03Z","comments":1,"url":"https://api.github.com/repos/intridea/omniauth-ldap/issues/2","body":"In omniauth-ldap/lib/omniauth/strategies/ldap.rb on line 60 extra information for auth_hash is provided.\r\n\r\nIn my case, and I would guess in general, it doesn't work since the @ldap_user_info is an instance of the Net::LDAP::Entry and doesn't fit in omniauth expectations from the return value.\r\n\r\nI fixed it by simply wrapping it in a hash. Providing the pull request together with the issue.","html_url":"https://github.com/intridea/omniauth-ldap/issues/2","title":"Handling of extra information within auth_hash raises exception for NET::LDAP instance","number":2,"id":2255278,"created_at":"2011-11-16T11:36:11Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/4f6fbd05e123ff2bb1d34de48424ddc1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"4f6fbd05e123ff2bb1d34de48424ddc1","url":"https://api.github.com/users/ivanjovanovic","login":"ivanjovanovic","id":23173},"updated_at":"2011-11-16T11:39:03Z","url":"https://api.github.com/repos/intridea/omniauth-ldap/issues/comments/2758508","body":"I'll close it and move explanation to the pull request","id":2758508,"created_at":"2011-11-16T11:39:03Z"}}