Closed
Bug 1222013
Opened 9 years ago
Closed 9 years ago
Add new FxA "devices API" error codes to FxAccountsCommon.js
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
DUPLICATE
of bug 1227527
People
(Reporter: stomlinson, Assigned: stomlinson)
References
Details
(Whiteboard: [fxa])
The new FxA "devices" API [1] will define several new error codes that can be returned when interacting with the FxA auth server endpoints. These error codes will need to be added to FxAccountsCommon.js [2]
[1] - https://github.com/mozilla/fxa/blob/phil/fxa-45-requirements/features/FxA-45-device-registration-api/README.md
[2] - https://dxr.mozilla.org/mozilla-central/source/services/fxaccounts/FxAccountsCommon.js#249
Assignee | ||
Comment 1•9 years ago
|
||
Phil, can you let me know whenever you have defined the new error codes?
Flags: needinfo?(pbooth)
Assignee | ||
Updated•9 years ago
|
Comment 2•9 years ago
|
||
Sorry for taking so long to get round to this. There are two new errors:
AppError.unknownDevice = function () {
return new AppError({
code: 400,
error: 'Bad Request',
errno: 123,
message: 'Unknown device'
})
}
AppError.deviceSessionConflict = function () {
return new AppError({
code: 400,
error: 'Bad Request',
errno: 124,
message: 'Session already registered by another device'
})
}
Flags: needinfo?(pbooth)
Comment 3•9 years ago
|
||
Actually, I think there may be one more too:
AppError.deviceSessionConflict = function () {
return new AppError({
code: 400,
error: 'Bad Request',
errno: 125,
message: 'Session already registered by another device'
})
}
Comment 4•9 years ago
|
||
Wait, I already mentioned that one. Ignore my last comment, it's Friday, late and I'm tired. :-/
Updated•9 years ago
|
Whiteboard: [fxa]
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•