1081023 - Adjust standalone link clicker UI to handle change of URL format.

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect)

Description

[Mark Banner (:standard8)]

Bug 1046114 is going to change the url format that the server provides. We need to update the standalone UI to handle the new format, currently it relies on:

#call/<token>

Comment 1

[Mark Banner (:standard8)]

Attachment: Handle call url changes to the format for Loop's call links

The main changes here are to the url handling in webapp.jsx. Most of the rest of the changes are consequential.

This makes it so that the standalone app can support urls of the format:

<hostname>/#call/{token}
<hostname>/c/{token}

as per bug 1046114 comment 33. It does not support the rooms format yet, as we haven't started the rooms work for standalone.

The changes in the server were mainly to try and be more explicit about which directories are hosted where on the server, to make maintenance easier. I also tried to document it, to make it easier in future.

Note that the old #call format will go away 30 days or so once we've transitioned the servers to the new format.

Comment 2

[Mark Banner (:standard8)]

Background for the regexps:

https://github.com/mozilla-services/loop-server/blob/master/loop/tokenlib.js#L10

Comment 3

[Mark Banner (:standard8)]

Attachment: Handle call url changes to the format for Loop's call links

Updated patch to account for bug 1079811 landing.

Setting review to NiKo or dmose.

Comment 4

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

Comment on attachment 8507195 [details] [diff] [review]
Handle call url changes to the format for Loop's call links

Review of attachment 8507195 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good but I feel like there's room for a tiny refactoring. As you're on PTO for 3 days, I'll take care of updating the patch.

::: browser/components/loop/standalone/content/js/webapp.js
@@ +929,1 @@
>      }

This should be extracted to some helper function, eg. extractLoopToken(document.location).

::: browser/components/loop/standalone/content/js/webapp.jsx
@@ +929,1 @@
>      }

How about moving this to some helper function so we could simply write:

var token = extractTokenFromLoopURL(document.location);
if (token) {
  conversation.set({loopToken: token});
}

Comment 5

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

Grmbl splinter. Oh well, you get the picture.

Comment 6

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

Comment on attachment 8506888 [details] [diff] [review]
Handle call url changes to the format for Loop's call links

Review of attachment 8506888 [details] [diff] [review]:
-----------------------------------------------------------------

Canceling review for this obsolete version of the patch.

Comment 7

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

Comment on attachment 8507195 [details] [diff] [review]
Handle call url changes to the format for Loop's call links

Review of attachment 8507195 [details] [diff] [review]:
-----------------------------------------------------------------

I'll actually r+ this, as working on a router impl. is being done in bug 1086588.

Comment 8

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=c0aa34523ce8
https://tbpl.mozilla.org/?tree=Try&rev=c0aa34523ce8

Comment 9

[Nicolas Perriault (:NiKo`) — needinfo me if you need my attention]

Removing checking-needed because of oranges: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=c0aa34523ce8

Seems unrelated to current patch, but I'm no expert in this area, hence why removing checkin-needed. Any help appreciated.

Comment 10

[Mark Banner (:standard8)]

Try issues were unrelated (random failures), so pushed the patch:

https://hg.mozilla.org/integration/fx-team/rev/6f8bcb8400a5

Comment 11

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/6f8bcb8400a5

Comment 12

[Randell Jesup [:jesup] (needinfo me)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/7203a9983dfc

Comment 13

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 8507195 [details] [diff] [review]
Handle call url changes to the format for Loop's call links

Approval Request Comment
Landed on aurora per IRC with lsblakk with a=loop-only

Comment 14

[Mark Banner (:standard8)]

This is covered by unit tests and functional tests.