{"action":"created","issue":{"url":"https://api.github.com/repos/knaperek/djangosaml2/issues/31","repository_url":"https://api.github.com/repos/knaperek/djangosaml2","labels_url":"https://api.github.com/repos/knaperek/djangosaml2/issues/31/labels{/name}","comments_url":"https://api.github.com/repos/knaperek/djangosaml2/issues/31/comments","events_url":"https://api.github.com/repos/knaperek/djangosaml2/issues/31/events","html_url":"https://github.com/knaperek/djangosaml2/pull/31","id":179328344,"number":31,"title":"Allow extra query args to be passed to the user model during auth.","user":{"login":"jamaalscarlett","id":7093153,"avatar_url":"https://avatars.githubusercontent.com/u/7093153?v=3","gravatar_id":"","url":"https://api.github.com/users/jamaalscarlett","html_url":"https://github.com/jamaalscarlett","followers_url":"https://api.github.com/users/jamaalscarlett/followers","following_url":"https://api.github.com/users/jamaalscarlett/following{/other_user}","gists_url":"https://api.github.com/users/jamaalscarlett/gists{/gist_id}","starred_url":"https://api.github.com/users/jamaalscarlett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamaalscarlett/subscriptions","organizations_url":"https://api.github.com/users/jamaalscarlett/orgs","repos_url":"https://api.github.com/users/jamaalscarlett/repos","events_url":"https://api.github.com/users/jamaalscarlett/events{/privacy}","received_events_url":"https://api.github.com/users/jamaalscarlett/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-26T20:25:34Z","updated_at":"2016-09-30T15:30:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/knaperek/djangosaml2/pulls/31","html_url":"https://github.com/knaperek/djangosaml2/pull/31","diff_url":"https://github.com/knaperek/djangosaml2/pull/31.diff","patch_url":"https://github.com/knaperek/djangosaml2/pull/31.patch"},"body":"SAML_DJANGO_USER_MAIN_ATTRIBUTE_FILTERS is a dictory of query args\r\npassed to the user model during backends.authenticate.\r\n\r\nUse case is a user model that contains a disabled field that would\r\nprevent access to the SP despite successful validation on the IDP.\r\n\r\nex. SAML_DJANGO_USER_MAIN_ATTRIBUTE_FILTERS = {'disabled': False}"},"comment":{"url":"https://api.github.com/repos/knaperek/djangosaml2/issues/comments/250775491","html_url":"https://github.com/knaperek/djangosaml2/pull/31#issuecomment-250775491","issue_url":"https://api.github.com/repos/knaperek/djangosaml2/issues/31","id":250775491,"user":{"login":"knaperek","id":1131179,"avatar_url":"https://avatars.githubusercontent.com/u/1131179?v=3","gravatar_id":"","url":"https://api.github.com/users/knaperek","html_url":"https://github.com/knaperek","followers_url":"https://api.github.com/users/knaperek/followers","following_url":"https://api.github.com/users/knaperek/following{/other_user}","gists_url":"https://api.github.com/users/knaperek/gists{/gist_id}","starred_url":"https://api.github.com/users/knaperek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/knaperek/subscriptions","organizations_url":"https://api.github.com/users/knaperek/orgs","repos_url":"https://api.github.com/users/knaperek/repos","events_url":"https://api.github.com/users/knaperek/events{/privacy}","received_events_url":"https://api.github.com/users/knaperek/received_events","type":"User","site_admin":false},"created_at":"2016-09-30T15:30:42Z","updated_at":"2016-09-30T15:30:42Z","body":"Is there not any nicer way achieve this? I wouldn't call this approach particularly clean since it only filters on certain _not-so-obviously-related_ condition.\r\n\r\nMoreover, I imagine you'd want to have a certain specific logging in case of _otherwise successful IdP assertion_ rejecting user just because of this. Now it stays hidden and may cause debugging troubles in the future."}}
{"actor":{"display_login":"knaperek"}}