JPakeClient.java onRequestSuccess exception block handling

RESOLVED FIXED in mozilla13

Status

()

P3
normal
RESOLVED FIXED
7 years ago
a year ago

People

(Reporter: dchanm+bugzilla, Assigned: nalexander)

Tracking

unspecified
mozilla13
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

(Reporter)

Description

7 years ago
The exception handling in onRequestSuccess seems inconsistent. Certain exceptions will cause an abort() in some parts of the code but not others [1]

A NonObjectJSONException from
800        jIncoming = response.jsonObjectBody();
will cause an abort but the same exception from
821          if (verifyPairing(jIncoming, myKeyBundle)) {
will print a stack trace.


What defines when an exception is fatal?


[1] - https://github.com/mozilla-services/android-sync/blob/develop/src/main/java/org/mozilla/gecko/sync/jpake/JPakeClient.java#L791
(Reporter)

Updated

7 years ago
Blocks: 723230

Updated

7 years ago
Assignee: nobody → liuche
(Assignee)

Updated

7 years ago
Assignee: liuche → nalexander
Priority: -- → P3
https://hg.mozilla.org/services/services-central/rev/6f1239da8d0e
Status: NEW → ASSIGNED
Whiteboard: [fixed in services][qa-]

Comment 4

7 years ago
https://hg.mozilla.org/mozilla-central/rev/92b773c99586
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13

Comment 5

7 years ago
https://hg.mozilla.org/mozilla-central/rev/6f1239da8d0e
Whiteboard: [fixed in services][qa-] → [qa-]
Component: Android Sync → Android Sync
Product: Mozilla Services → Android Background Services

Updated

a year ago
Product: Android Background Services → Firefox for Android
You need to log in before you can comment on or make changes to this bug.