523689 - add download source tracking for install service

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Administration, defect, P3)

Description

[Justin Scott [:fligtar]]

The install service should accept a src parameter that it passes to the file download URL so that these add-ons can use source tracking too.

So, https://addons.mozilla.org/services/install.php?addon_id=personas&src=external-getpersonas should add the src param to the xpi URL.

Comment 1

[Justin Scott [:fligtar]]

And if there is no src= parameter set, let's default to src=installservice and we'll need to add that as an allowed src in the db and add copy to the sources page:

From Install Service
Downloads passed through the AMO Install Service are counted here.

Comment 2

[Justin Scott [:fligtar]]

Attachment: patch, v1

Really want this in 5.3 so doing it myself!

Comment 3

[Wil Clouser [:clouserw]]

Comment on attachment 408750 [details] [diff] [review]
patch, v1

You might want to explain what the AMO Install Service is.

Comment 4

[Justin Scott [:fligtar]]

I was intentionally vague as its existence is not well-known. I may add a little more detail.

Comment 5

[Justin Scott [:fligtar]]

r54493

Comment 6

[Justin Scott [:fligtar]]

I forgot to do SQL for this bug!

Comment 7

[Justin Scott [:fligtar]]

eh, I'll just do it in bug 526600

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified, as this is being tracked/verified in bug 526600.