link-clicker calls using dev-server don't work

RESOLVED WORKSFORME

Status

P1
major
RESOLVED WORKSFORME
4 years ago
4 years ago

People

(Reporter: dmose, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa+])

Setting loop.server to https://loop-dev.stage.mozaws.net and then sharing a link, fails to connect a call.

The shared link loads the "start call" page as usual, albeit there are a few interesting warnings in the browser console:

TypeError: loop.config.fxosApp is undefined config.js:7
13:01:54.433 Security wrapper denied access to property length on privileged Javascript object. Support for exposing privileged objects to untrusted content via __exposedProps__ is being gradually removed - use WebIDL bindings or Components.utils.cloneInto instead. Note that only the first denied property access from a given global object will be reported.

Clicking start shows "Connecting..." for a short while (10 secs?), then switches to call failed.

The link generator browser never sees any visible evidence of an incoming call.

Standard8 was looking at something that made him think that the latest server bits were somehow not triggering the necessary push notifications.

It's certainly the case that running 

env LOOP_SERVER=/Users/dmose/r/loop-server mach marionette-test browser/components/loop/test/functional/manifest.ini 

with a copy of the latest server in the pointed to directory fails.
The way we handle SimplePushURLs changed with this version, make sure that you've made a POST /registration with this token since you've changed the server.
(Reporter)

Comment 2

4 years ago
I'm confused.  Shouldn't such a post happen at every browser startup?  If not, how do I force it?
(In reply to Rémy Hubscher (:natim) from comment #1)
> The way we handle SimplePushURLs changed with this version, make sure that
> you've made a POST /registration with this token since you've changed the
> server.

We are doing the same POST /registration that we always have, with just using simplePushURL as the parameter. According to the docs, this is still supported:

https://wiki.mozilla.org/Loop/Architecture/Rooms#Loop_Server_API_Changes

I haven't looked at what changes have been made to the server, so this may be irrelevant.

I have verified the clients work with 0.12.4, but not with latest master.
Okay, we're looking at it.
Severity: normal → major
Priority: -- → P1
We should check Stage as well...
Status: NEW → ASSIGNED
Whiteboard: [qa+]
I am pretty sure 0.12.6 has got this bug as well.
I don't think this problem still exists, but if it's the case, I'm not 100% sure to undestand what it is.

Can you confirm that when you're doing a call from Bob to Alice using the link-clicker UI, Bob's client isn't waken up via simple push?
Flags: needinfo?(standard8)
And of course, it's wrong. Let me do it again:

Can you confirm that when you're doing a call from Bob to Alice using the link-clicker UI, Alice's client isn't waken up via simple push?
I've just been trying this on dev and its working fine. I've done a combination of direct calls and link clicker calls both ways.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Flags: needinfo?(standard8)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.