Closed
Bug 1382561
Opened 7 years ago
Closed 7 years ago
Sync does not start after email confirmation
Categories
(Firefox for iOS :: Sync, defect, P1)
Tracking
()
People
(Reporter: SimonB, Assigned: jhugman)
References
Details
(Keywords: regression, Whiteboard: [MobileCore][Sync])
Attachments
(1 file)
Build: e14617f7e3
Device: iPad Air 2
iOS: 10.3.3
Steps to reproduce:
1. Sign in to FxA on iOS Device
2. Go to inbox and confirm the sign in
Actual results:
- After confirming the sing in, sync does not start automatically.
Expected result:
- Sync should be triggered after confirming the sing in.
|
Reporter | |
Updated•7 years ago
|
Keywords: regression
|
||
Updated•7 years ago
|
|
||
Updated•7 years ago
|
Whiteboard: [MobileCore][Sync]
|
|
||
Updated•7 years ago
|
|
|
||
Updated•7 years ago
|
|
Assignee | |
Comment 1•7 years ago
|
||
From Bug 1376007, commment 3:
> Last week worked with James and this is what we found/updated:
>
> * Deployed a new version of http://ios-push2.dev.lcip.org, that always sends an `accountVerify` message.
> * Changed `case accountVerified = "accountVerify"`, line 224 in FxAPushMessageHandler
>
> Tested with an updated PyFxA script to simulate sending the message
>
> ```
> from getpass import getpass
> from fxa.core import Client
>
> c = Client("https://ios-push2.dev.lcip.org/auth/v1")
> s = c.login(email, password)
>
> try:
> s.apiclient.post("/account/devices/notify", json={
> "to": "all",
> "endpointAction": "accountVerify",
> "payload": {}
> }, auth=s._auth)
> finally:
> s.destroy_session()
> ```
>
> I noticed that if the account is unverified, the push messaged received is nil. Manually verifying the account and re-running script correctly shows an `accountVerify` push message. Is it possible that there is something in the iOS client that doesn't correctly register push messaging until after an account has been confirmed?
|
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → jhugman
|
|
Assignee | |
Updated•7 years ago
|
|
|
Assignee | |
Updated•7 years ago
|
Iteration: --- → 1.30
Priority: -- → P1
|
|
Assignee | |
Updated•7 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•7 years ago
|
||
Attachment #8908264 -
Flags: review?(sarentz)
Attachment #8908264 -
Flags: review?(jdarcangelo)
|
||
Updated•7 years ago
|
Attachment #8908264 -
Flags: review?(jdarcangelo) → review?(rnewman)
|
|
Assignee | |
Comment 3•7 years ago
|
||
Comment on attachment 8908264 [details]
Link to Github pull-request: https://github.com/mozilla-mobile/firefox-ios/pull/3177
Removing from st3fan 's reviews.
Attachment #8908264 -
Flags: review?(sarentz)
|
||
Updated•7 years ago
|
|
|
Assignee | |
Updated•7 years ago
|
Attachment #8908264 -
Flags: review?(rnewman) → review?(jdarcangelo)
|
|
||
Updated•7 years ago
|
Attachment #8908264 -
Flags: review?(jdarcangelo) → review+
|
|
Assignee | |
Comment 4•7 years ago
|
||
Merged into master.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [MobileCore][Sync] → [MobileCore][Sync][NeedsUplift]
|
|
Reporter | |
Comment 5•7 years ago
|
||
Verifying as fix on master a9b288f1c85a.
status-fxios-v10.0:
--- → verified
|
|
||
Comment 6•7 years ago
|
||
Uplifted to v10.x
Whiteboard: [MobileCore][Sync][NeedsUplift] → [MobileCore][Sync]
You need to log in
before you can comment on or make changes to this bug.
Description
•