{"action":"opened","issue":{"url":"https://api.github.com/repos/openshift/oauth-proxy/issues/83","repository_url":"https://api.github.com/repos/openshift/oauth-proxy","labels_url":"https://api.github.com/repos/openshift/oauth-proxy/issues/83/labels{/name}","comments_url":"https://api.github.com/repos/openshift/oauth-proxy/issues/83/comments","events_url":"https://api.github.com/repos/openshift/oauth-proxy/issues/83/events","html_url":"https://github.com/openshift/oauth-proxy/issues/83","id":352093978,"node_id":"MDU6SXNzdWUzNTIwOTM5Nzg=","number":83,"title":"'authenticated-emails-file' option not respected by default","user":{"login":"thirdeyenick","id":10754194,"node_id":"MDQ6VXNlcjEwNzU0MTk0","avatar_url":"https://avatars3.githubusercontent.com/u/10754194?v=4","gravatar_id":"","url":"https://api.github.com/users/thirdeyenick","html_url":"https://github.com/thirdeyenick","followers_url":"https://api.github.com/users/thirdeyenick/followers","following_url":"https://api.github.com/users/thirdeyenick/following{/other_user}","gists_url":"https://api.github.com/users/thirdeyenick/gists{/gist_id}","starred_url":"https://api.github.com/users/thirdeyenick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thirdeyenick/subscriptions","organizations_url":"https://api.github.com/users/thirdeyenick/orgs","repos_url":"https://api.github.com/users/thirdeyenick/repos","events_url":"https://api.github.com/users/thirdeyenick/events{/privacy}","received_events_url":"https://api.github.com/users/thirdeyenick/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-08-20T11:25:25Z","updated_at":"2018-08-20T11:25:25Z","closed_at":null,"author_association":"NONE","body":"Hi,\r\nI just tested the oauth-proxy in a simple application. I used the --authenticated-emails-file option. After testing access I noticed that all email addresses are allowed, so you just need to have a valid openshift account. Although this is documented I still think that just passing the 'authenticated-emails-file' should give the expected result of just allowing the given addresses.\r\n\r\nI had a look into the code and found that by default all domains ('*') are allowed, not respecting any given 'authenticated-emails-file' option (options.go line 165). Maybe we could check there if an authenticated-emails-file option was given and if yes just do a\r\n```golang\r\no.EmailDomains = []string{}\r\n```\r\n?"}}
{"actor":{"display_login":"thirdeyenick"}}