If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[Loop] Sign up requests might be unauthorized due to assertion certificate expired

RESOLVED FIXED

Status

Firefox OS
Gaia::Loop
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jaoo, Assigned: jaoo)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mobile app][blocking][tef-triage][loop approved for 1.1][loop in v1.1])

Attachments

(4 attachments)

We are seeing 'WWW-Authenticate BrowserID error="expired"' server responses on sign up requests. Need to handle better how to inform the user on unauthorized sign up requests. Right now we just inform the user about it by using the generic server error dialog. This information would need to be improved.
(Assignee)

Updated

3 years ago
Blocks: 1036490
(Assignee)

Updated

3 years ago
Blocks: 1088993
(Assignee)

Updated

3 years ago
Assignee: nobody → josea.olivera
(Assignee)

Comment 1

3 years ago
Created attachment 8530936 [details] [review]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/348

Comment 2

3 years ago
Created attachment 8531563 [details]
UXPin-Loop_V1.1_No_Connection-FxA-Expired.pdf

This is the WF + Spec.
(Assignee)

Updated

3 years ago
Attachment #8530936 - Flags: review?(borja.bugzilla)
(Assignee)

Updated

3 years ago
Attachment #8530936 - Flags: review?(borja.bugzilla) → review?(amac.bug)
(Assignee)

Updated

3 years ago
Summary: [Loop] Sign up requests might be unauthorized due to assertions expired → [Loop] Sign up requests might be unauthorized due to assertion certificate expired
(Assignee)

Comment 3

3 years ago
Created attachment 8532015 [details]
Screenshot

Monica, would you mind to have a look at how the new screen looks please? The string has changed a bit. I hope you don't mind we to take this new string. Please, let us know. Thanks!
Attachment #8532015 - Flags: ui-review?(monica.meika)
Comment on attachment 8530936 [details] [review]
Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/348

I left a comment on Github. The log should be an error, and unconditional IMHO (since that's something that most definitely should never come up on the normal life of the app). Other than that, it looks great to me, thank you!

Change that (or not if you believe it's better behind a debug flag) and land it when you want.
Attachment #8530936 - Flags: review?(amac.bug) → review+

Comment 5

3 years ago
Comment on attachment 8532015 [details]
Screenshot

If we add "Please try logging in again"..
I would change the buttons: [Ok] [Retry], :)
Attachment #8532015 - Flags: ui-review?(monica.meika)

Updated

3 years ago
Attachment #8532015 - Flags: ui-review+
(Assignee)

Comment 6

3 years ago
Landed on master branch at:

https://github.com/mozilla-b2g/firefoxos-loop-client/commit/6e9a7ecd1406b7089d34769bcbfad918ca2ffaf8
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Whiteboard: [mobile app][blocking][tef-triage][loop approved for 1.1][loop not in v1.1]
(Assignee)

Comment 7

3 years ago
Created attachment 8532077 [details] [review]
[1.1] Pointer to Github PR https://github.com/mozilla-b2g/firefoxos-loop-client/pull/361

1.1. patch. Keeping r=amac
(Assignee)

Comment 8

3 years ago
Landed on 1.1 branch at:

https://github.com/mozilla-b2g/firefoxos-loop-client/commit/4212078dd18628028661e7cc7775924d718630e8
Whiteboard: [mobile app][blocking][tef-triage][loop approved for 1.1][loop not in v1.1] → [mobile app][blocking][tef-triage][loop approved for 1.1][loop in v1.1]
Blocks: 1112150
You need to log in before you can comment on or make changes to this bug.