philbooth · GitHub

Merged

Merged

Conversation

@philbooth

Copy link Copy Markdown

Contributor

Fixes #68.

We don't want to let knowledge about the auth db leak into any unrelated application logic, because it's both temporary and in possession of capabilities that this repo has no business knowing about.

@mozilla/fxa-devs r?

@philbooth

@vladikoff

@vladikoff

Copy link Copy Markdown

Contributor

brizental

Copy link Copy Markdown

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

vladikoff

Copy link Copy Markdown

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

R+, i like the new tests!

@vladikoff

@philbooth

Read the original on github.com ↗