This was referenced
May 28, 2018Merged
Merged
philbooth added a commit that referenced this pull request
May 30, 2018A direct mapping of functionality from the auth server, so the different notification types are expected on three different queues. Long-term it makes sense to simplify that down to a single queue, but this way we can ease the new service in alongside the existing auth server. As a first cut, the methods on queues::sqs::Queue all block the thread in this implementation. A subsequent change will transition to futures. #28 r=rfk
philbooth added a commit that referenced this pull request
May 30, 2018A direct mapping of functionality from the auth server, so the different notification types are expected on three different queues. Long-term it makes sense to simplify that down to a single queue, but this way we can ease the new service in alongside the existing auth server. As a first cut, the methods on queues::sqs::Queue all block the thread in this implementation. A subsequent change will transition to futures. #28 r=rfk
philbooth added a commit that referenced this pull request
May 30, 2018A direct mapping of functionality from the auth server, so the different notification types are expected on three different queues. Long-term it makes sense to simplify that down to a single queue, but this way we can ease the new service in alongside the existing auth server. As a first cut, the methods on queues::sqs::Queue all block the thread in this implementation. A subsequent change will transition to futures. #28 r=rfk
rfk approved these changes Jun 5, 2018
A direct mapping of functionality from the auth server, so the different notification types are expected on three different queues. Long-term it makes sense to simplify that down to a single queue, but this way we can ease the new service in alongside the existing auth server. As a first cut, the methods on queues::sqs::Queue all block the thread in this implementation. A subsequent change will transition to futures.