1064862 - Desktop client should cope with trailing slashes on loop.server pref

Status: RESOLVED WORKSFORME

(Hello (Loop) :: Client, defect, P5)

Description

[Mark Banner (:standard8)]

Coming out of the comments on bug 1032738 (bug 1032738 comment 15 and onwards), it appears one of the errors was due to the loop.server preference being set with a trailing slash.

We should consider, for developer sanity, allowing preferences with trailing slashes.

At the moment, this would affect both the MozLoopService and MozLoopPushHandler modules.

Comment 1

[Maire Reavy [:mreavy]]

Moving this to P2 based on our new priority definitions.

Comment 2

[Mark Banner (:standard8)]

For developers:

- The places to start looking are for the loop.server prefs:

http://mxr.mozilla.org/mozilla-central/search?string=loop.server

- Currently things break when we add a trailing slash onto the end of the pref, so adding that and testing the Hello button on the toolbar should reveal any issues.

Note that for the push handler, this is the push notification service, so you'll either need to test receiving calls from another browser, or test entering a room with another browser/machine.

Comment 3

[cedric raudin]

Hi,

Can I work on this bug?

Comment 4

[Mark Banner (:standard8)]

Yep, sure, assigning you to it.

Comment 5

[Mark Banner (:standard8)]

Cedric pointed out to me offline, that the original issue that caused this bug to be filed is no longer present.

I suspect the recent server updates have somehow fixed this.

Whilst its a bit messy to have double-slashes in urls, I don't think it really matters in this case as it isn't our default mode of operating.

Therefore, I'm going to close this as works for me as we don't need it now.

Comment 6

[Mark Banner (:standard8)]

and thanks to Cedric for testing it!