mozilla / fxa-auth-server

DEPRECATED - Migrated to https://github.com/mozilla/fxa
Mozilla Public License 2.0
399 stars 121 forks source link

feat(devices): devices API with refreshTokens #2937

Closed vladikoff closed 5 years ago

vladikoff commented 5 years ago

Connects to https://github.com/mozilla/fxa-auth-server/issues/2547 Fixes https://github.com/mozilla/fxa-auth-server/issues/2547 Fixes https://github.com/mozilla/fxa-auth-server/issues/2936

TODO:

Follow up issues

https://github.com/mozilla/fxa-auth-server/issues/2956 https://github.com/mozilla/fxa-auth-server/issues/2973 https://github.com/mozilla/fxa-auth-server/issues/2974 https://github.com/mozilla/fxa-auth-server/issues/2975

rfk commented 5 years ago

Some things to consider from the matching db-server PR [1]:

We may be able to defer some or all of these to follow-up work, but let's be explicit about it so none of them get lost.

[1] https://github.com/mozilla/fxa-auth-db-mysql/pull/483#issuecomment-467793239

vladikoff commented 5 years ago

When listing device records, we should also list all the refresh tokens on the oauth server and do a "merge" to eliminate zombie devices whose refresh token no longer exists.

Let's do a file up issue for this one

vladikoff commented 5 years ago

SendTab PR tester run it via: RUST_BACKTRACE=1 cargo run --example devices_api

vladikoff commented 5 years ago

@rfk this should be ready to go

rfk commented 5 years ago

(Also, watch out for github hiding some of my comments away behind its little "hidden conversations" toggle...)

vladikoff commented 5 years ago

@rfk All the requested changes are in https://github.com/mozilla/fxa-auth-server/pull/2937/commits/f748042a0a27897aedd1bdac13e943dca4c34f48 plus a fix for https://github.com/mozilla/fxa-auth-server/issues/2936 . Let me know!