Build a new endpoint serving android snippets JSON

VERIFIED FIXED

Status

VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: hoosteeno, Assigned: giorgos)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [kb=1187290] )

This endpoint will serve only json/android snippets that match the client-match rules in the request URL. It will serve all geographies and the client will sort out any geotargeted snippets.

Let's use this bug's comments to specify the actual URL this endpoint will live at.
(Assignee)

Comment 1

5 years ago
Current snippet url config lives under

browser.aboutHomeSnippets.updateUrl

and has a default value of

https://snippets.mozilla.com/%STARTPAGE_VERSION%/%NAME%/%VERSION%/%APPBUILDID%/%BUILD_TARGET%/%LOCALE%/%CHANNEL%/%OS_VERSION%/%DISTRIBUTION%/%DISTRIBUTION_VERSION%/


Margaret:

Do you plan to use the same configuration variable (updateUrl) or you'll be adding a new one?

I'm asking so we can decide whether we will have to use the same endpoint for Android and depend on the %NAME% change to serve json and the Android snippets

OR

we are given the chance to create a new similar endpoint like for example:

https://snippets.mozilla.com/json/%STARTPAGE_VERSION%/%NAME%/%VERSION%/%APPBUILDID%/%BUILD_TARGET%/%LOCALE%/%CHANNEL%/%OS_VERSION%/%DISTRIBUTION%/%DISTRIBUTION_VERSION%/

(Note the "json" part at the start)

Thanks!
Flags: needinfo?(margaret.leibovic)
(Assignee)

Comment 2

5 years ago
Got the answer on IRC:

<mkelly> giorgos: margaret said they're using a seperate pref
<mkelly> So we can set the URL to whatever
Flags: needinfo?(margaret.leibovic)

Comment 3

5 years ago
If you want to test, I put a test build with the patch from bug 937820 here (it will show up as fennec_leibovic in your app launcher):
https://dl.dropboxusercontent.com/u/3358452/fennec-snippets.apk

Here's the pref that patch uses:
pref("browser.snippets.updateUrl", "https://snippets.mozilla.com/json/%SNIPPETS_VERSION%/%NAME%/%VERSION%/%APPBUILDID%/%BUILD_TARGET%/%LOCALE%/%CHANNEL%/%OS_VERSION%/%DISTRIBUTION%/%DISTRIBUTION_VERSION%/");

I used a %SNIPPETS_VERSION% variable name instead of %STARTPAGE_VERSION%, but the idea should be the same. Right now I just have it set to 1. So the URL this build actually uses is:
https://snippets.mozilla.com/json/1/Fennec/28.0a1/20131118093510/Android_arm-eabi-gcc3/en-US/default/Linux%2018/default/default/

Feel free to go into about:config to change the pref to a development URL.

(Also, FYI, that build just uses a hardcoded "US" country code, and it doesn't ping any metrics server.)
Whiteboard: [kb=1187290]
(Assignee)

Updated

5 years ago
Assignee: nobody → giorgos
Status: NEW → ASSIGNED
Blocks: 945880
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED

Updated

5 years ago
Blocks: 962349
You need to log in before you can comment on or make changes to this bug.