{"action":"created","issue":{"url":"https://api.github.com/repos/akka/akka/issues/17643","labels_url":"https://api.github.com/repos/akka/akka/issues/17643/labels{/name}","comments_url":"https://api.github.com/repos/akka/akka/issues/17643/comments","events_url":"https://api.github.com/repos/akka/akka/issues/17643/events","html_url":"https://github.com/akka/akka/pull/17643","id":84443357,"number":17643,"title":"per #17384 - Generalised Security Directives to accept Bearer Token Authentication","user":{"login":"pjan","id":98241,"avatar_url":"https://avatars.githubusercontent.com/u/98241?v=3","gravatar_id":"","url":"https://api.github.com/users/pjan","html_url":"https://github.com/pjan","followers_url":"https://api.github.com/users/pjan/followers","following_url":"https://api.github.com/users/pjan/following{/other_user}","gists_url":"https://api.github.com/users/pjan/gists{/gist_id}","starred_url":"https://api.github.com/users/pjan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pjan/subscriptions","organizations_url":"https://api.github.com/users/pjan/orgs","repos_url":"https://api.github.com/users/pjan/repos","events_url":"https://api.github.com/users/pjan/events{/privacy}","received_events_url":"https://api.github.com/users/pjan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-03T08:53:14Z","updated_at":"2015-06-03T08:56:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/akka/akka/pulls/17643","html_url":"https://github.com/akka/akka/pull/17643","diff_url":"https://github.com/akka/akka/pull/17643.diff","patch_url":"https://github.com/akka/akka/pull/17643.patch"},"body":"As per promise to @rkuhn, an improved shot to allow Bearer Token Authentication, taking into accounts the comments made by @jrudolph in #17384, post the #17502 merge. Since the `OAuth2BearerToken` is part of the akka codebase, I opted to add support for it within the http module rather than provide additional facilities to provide it as a third party library.\r\n\r\n- Renamed `UserCredentials` to the more general `Credentials`, and its `userName` param to the more appropriate `identifier` in case it covers both `HttpBasicAuthentication` and `BearerTokenAuthentication`.\r\n- Dropped `Basic` from the directives, as they now similarly accept `Authenticator`s that handle `BasicHttpCredentials` and `OAuth2BearerToken`\r\n- Added and amended the tests for both scala and java side"},"comment":{"url":"https://api.github.com/repos/akka/akka/issues/comments/108252637","html_url":"https://github.com/akka/akka/pull/17643#issuecomment-108252637","issue_url":"https://api.github.com/repos/akka/akka/issues/17643","id":108252637,"user":{"login":"akka-ci","id":2511948,"avatar_url":"https://avatars.githubusercontent.com/u/2511948?v=3","gravatar_id":"","url":"https://api.github.com/users/akka-ci","html_url":"https://github.com/akka-ci","followers_url":"https://api.github.com/users/akka-ci/followers","following_url":"https://api.github.com/users/akka-ci/following{/other_user}","gists_url":"https://api.github.com/users/akka-ci/gists{/gist_id}","starred_url":"https://api.github.com/users/akka-ci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akka-ci/subscriptions","organizations_url":"https://api.github.com/users/akka-ci/orgs","repos_url":"https://api.github.com/users/akka-ci/repos","events_url":"https://api.github.com/users/akka-ci/events{/privacy}","received_events_url":"https://api.github.com/users/akka-ci/received_events","type":"User","site_admin":false},"created_at":"2015-06-03T08:56:02Z","updated_at":"2015-06-03T08:56:02Z","body":"Can one of the repo owners verify this patch?"}}