philbooth · GitHub

@philbooth

philbooth

philbooth

philbooth

philbooth

philbooth

philbooth

philbooth

philbooth

philbooth

philbooth

@philbooth

@philbooth

rfk

This was referenced

May 28, 2018

Merged

Merged

@philbooth

philbooth added a commit that referenced this pull request

May 30, 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.
#28
r=rfk

@philbooth

philbooth added a commit that referenced this pull request

May 30, 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.
#28
r=rfk

@philbooth

philbooth added a commit that referenced this pull request

May 30, 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.
#28
r=rfk

@philbooth

@philbooth

rfk

rfk approved these changes Jun 5, 2018

@philbooth

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.

@philbooth

email address validation is a pain, and you can use a surprising variety
of characters in an address.
(At least we're drawing the line at no quoted characters...)
Closes #44
#45
r=philbooth

@philbooth

@philbooth

@philbooth

Closed

Read the original on github.com ↗