Closed
Bug 1079950
Opened 11 years ago
Closed 11 years ago
GET /calls/?version=<version> should include the callerId for both FxA and non-FxA calls
Categories
(Hello (Loop) :: Server, defect, P1)
Hello (Loop)
Server
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: standard8, Assigned: rhubscher)
References
Details
(Whiteboard: [qa+])
Attachments
(1 file)
In implementing bug 1020449, I've noticed that in the response from GET /calls/?version=<version> for FxA calls (direct fxa-fxa), we're not getting the "callerId" specified.
It is in the non-FxA link clicker calls.
For FxA calls, the callerId should be set to the FxA id of the caller.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 2•11 years ago
|
||
I have added a test, if the UserId is present GET /calls returns the callerId.
I will investigate this afternoon to make sure FxA Oauth set the UserId correctly at login.
Assignee | ||
Updated•11 years ago
|
Attachment #8502374 -
Flags: review?(alexis+bugs)
Assignee | ||
Comment 3•11 years ago
|
||
I found and fixed the bug, thank you Mark for finding it.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rhubscher
Status: NEW → ASSIGNED
Comment 4•11 years ago
|
||
Comment on attachment 8502374 [details] [review]
Link to Github PR — #224.
yep, all good! https://github.com/mozilla-services/loop-server/commit/b5f5857cd0b224307653bab972a0c098ff121ec1
Attachment #8502374 -
Flags: review?(alexis+bugs) → review+
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Whiteboard: [qa+]
Comment 5•11 years ago
|
||
Alexis, Tarek -- Is this getting deployed this week or early next? This enables caller ID and call blocking which we have to have in Fx34 by next week. Alexis and I talked about it early today (roughly 8 hours ago), and my understanding was that it should go out in a few days -- no later than early next week. Am I correct?
In the meantime, I need to make sure we have a loop server to test against so that client side changes can be tested, verified & uplifted. I want to uplift the client code into Beta and be ready for when the server fix goes live. Thanks.
Flags: needinfo?(tarek)
Flags: needinfo?(alexis+bugs)
Comment 6•11 years ago
|
||
Per IRC discussions, this appears to be needed in the same timeframe as 0.12.4, which is currently in Stage.
Can this be fixed overnight and rebuilt/deployed to Stage soon after that?
Or do we want to try this for next week?
(timing is everything re: Fx34 and beyond)
Severity: normal → major
Priority: -- → P1
Comment 7•11 years ago
|
||
Maire, we need this to ride the trains, at the latest, so it will need to go trough QA before being deployed.
From what we said earlier, you said it would be okay to have this in 6 weeks time.
From IRC logs:
14:24 <mreavy> alexis, mikedeboer : if I explicit select "block contact" from the UI, that contact should not be able to call me. (It's a privacy requirement.)
14:25 <mreavy> alexis, mikedeboer : it's not a requirement for getting into beta, but it is a requirement for going to release in 6 weeks.
14:25 <@alexis> mreavy: We don't have anything related to that on the server side I believe, do we?
14:25 <mreavy> alexis: it relies on caller ID
14:26 <mreavy> alexis: if we don't get the correct caller ID from the server, we don't know to block the call
14:26 <mreavy> mikedeboer: correct me if I'm incorrect ^^^
14:26 <@alexis> e.g. we don't have any blacklist on the serverside
14:26 <@alexis> I thought the block contact would be done only on the client side
14:27 <mikedeboer> mreavy: yes, indeed. But what the server bug does effectively is give us the Fx Account ID, which the client doesn't know what to do with
14:27 <mreavy> alexis: it is on the client side, but we rely on the server giving us the Fx acct ID
14:27 <mreavy> alexis: what mikedeboer just said ;-)
14:27 <@alexis> yeah okay, I get it now
14:27 <@alexis> it will be deployed soon enough :)
14:28 <mreavy> alexis: cool, thanks. Just wanted to let you know it's not a "nice to have" for release in 6 weeks. It's one of the few things that privacy said must be working before we release
Also, it needs action from dev to tag and then action from ops to deploy to stage, then be verified by QA. This needs normally a few days, but QA will be able to tell more.
Flags: needinfo?(alexis+bugs)
Comment 8•11 years ago
|
||
(Clearing NI for tarek along the way)
Okay, after some more discussions with Maire it seems that there is a big misunderstanding, and we need this to get out really quickly.
I tagged a new version which should go out instead of the current train we had. It's bug 1082919.
Flags: needinfo?(tarek)
Comment 9•11 years ago
|
||
It's out there. Version 0.12.5 is in Stage and in Prod.
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 10•11 years ago
|
||
Note to testers: for a callee to receive the caller id, the caller must have last signed into FxA after the deployment time of 0.12.5.
You need to log in
before you can comment on or make changes to this bug.
Description
•