1026426 - Handle short URLS

Status: VERIFIED FIXED

(Hello (Loop) :: Server, defect, P1)

Description

[Alexis Metaireau (:alexis)]

Call URLs should be shorter, and we should store data on the server side, not directly inside the token anymore.

Comment 2

[Maire Reavy [:mreavy]]

Alexis, Remy -- How quickly do you think you're going to get to this (relative to your other work/priorities)?  This isn't blocking our work, but many users are asking for shorter, more user-friendly URLs as soon as possible.  Thanks.

Comment 3

[Rémy Hubscher (:natim)]

It will land with loop-server MVP.

Comment 4

[Maire Reavy [:mreavy]]

(In reply to Rémy Hubscher (:natim) from comment #3)
> It will land with loop-server MVP.

To confirm: Is that by July 7th?

Comment 5

[Tarek Ziadé (:tarek)]

As per yesterday meeting : https://wiki.mozilla.org/CloudServices/Loop/WeeklyStandup/2014-06-17

* July 7th is latest date we truly need staging server to test Loop Server features for FxOS changes
* VALIDATE: July 21st is when we need production server for desktop client - Shell validate!!! if we need in production by July 21st or later.....

Comment 6

[Alexis Metaireau (:alexis)]

I believe we can leave with this for the time being. MVP will ship with this, for sure. Please comment if you need it sooner that that, but timelines are already pretty hard.

Comment 7

[Tarek Ziadé (:tarek)]

Adding back the original Priority/Target from bug 1026764

Comment 8

[Rémy Hubscher (:natim)]

Attachment: Link to github PR

Comment 9

[Romain Testard [:RT]]

More info on link length provided by abr on https://bugzilla.mozilla.org/show_bug.cgi?id=1000770#c21

https://wiki.mozilla.org/Loop/Architecture/MVP#Token_Length_and_Generation

Comment 10

[Rémy Hubscher (:natim)]

Thanks you Romain, we used this to let the user configure this length in bytes rather than in caracters so you can calculate the best length based on this calculation automatically.

Comment 11

[Alexis Metaireau (:alexis)]

https://github.com/mozilla-services/loop-server/commit/185cfb0112ebecf5b5b52af2658a4ec1697b5ca6

Comment 12

[Alexis Metaireau (:alexis)]

Comment on attachment 8447140 [details] [review]
Link to github PR

Added a few changes here and there in the PR, thanks!

Comment 13

[James Bonacci [:jbonacci]]

Verified in code and in unit tests.