{"action":"created","issue":{"url":"https://api.github.com/repos/99designs/aws-vault/issues/92","repository_url":"https://api.github.com/repos/99designs/aws-vault","labels_url":"https://api.github.com/repos/99designs/aws-vault/issues/92/labels{/name}","comments_url":"https://api.github.com/repos/99designs/aws-vault/issues/92/comments","events_url":"https://api.github.com/repos/99designs/aws-vault/issues/92/events","html_url":"https://github.com/99designs/aws-vault/pull/92","id":178554789,"number":92,"title":"Use iam credentials for login, allowing session expiry of 12h","user":{"login":"lox","id":15758,"avatar_url":"https://avatars.githubusercontent.com/u/15758?v=3","gravatar_id":"","url":"https://api.github.com/users/lox","html_url":"https://github.com/lox","followers_url":"https://api.github.com/users/lox/followers","following_url":"https://api.github.com/users/lox/following{/other_user}","gists_url":"https://api.github.com/users/lox/gists{/gist_id}","starred_url":"https://api.github.com/users/lox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lox/subscriptions","organizations_url":"https://api.github.com/users/lox/orgs","repos_url":"https://api.github.com/users/lox/repos","events_url":"https://api.github.com/users/lox/events{/privacy}","received_events_url":"https://api.github.com/users/lox/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-09-22T08:52:27Z","updated_at":"2016-09-22T08:55:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/99designs/aws-vault/pulls/92","html_url":"https://github.com/99designs/aws-vault/pull/92","diff_url":"https://github.com/99designs/aws-vault/pull/92.diff","patch_url":"https://github.com/99designs/aws-vault/pull/92.patch"},"body":"Based on feedback in this [aws forum thread](https://forums.aws.amazon.com/thread.jspa?messageID=742905&#742905), it became obvious that the new longer federated session expiry could be used if IAM credentials were used to assume the role, vs our standard iam -> session token -> assume role -> federation tokens. \r\n\r\nI'm not sure why I never realized that the session token step was unneeded. I suspect it was because it adds complexity to the code, as you need to conditionally apply MFA to the AssumeRole call if there isn't a session with MFA already applied to it.\r\n\r\nRegardless, this works, although I've been drinking a lot of wine, so I might be imagining it all. "},"comment":{"url":"https://api.github.com/repos/99designs/aws-vault/issues/comments/248846812","html_url":"https://github.com/99designs/aws-vault/pull/92#issuecomment-248846812","issue_url":"https://api.github.com/repos/99designs/aws-vault/issues/92","id":248846812,"user":{"login":"lox","id":15758,"avatar_url":"https://avatars.githubusercontent.com/u/15758?v=3","gravatar_id":"","url":"https://api.github.com/users/lox","html_url":"https://github.com/lox","followers_url":"https://api.github.com/users/lox/followers","following_url":"https://api.github.com/users/lox/following{/other_user}","gists_url":"https://api.github.com/users/lox/gists{/gist_id}","starred_url":"https://api.github.com/users/lox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lox/subscriptions","organizations_url":"https://api.github.com/users/lox/orgs","repos_url":"https://api.github.com/users/lox/repos","events_url":"https://api.github.com/users/lox/events{/privacy}","received_events_url":"https://api.github.com/users/lox/received_events","type":"User","site_admin":false},"created_at":"2016-09-22T08:55:47Z","updated_at":"2016-09-22T08:55:47Z","body":"Fixes #77. "}}
{"actor":{"display_login":"lox"}}