Opened 2 years ago

Closed 17 months ago

Last modified 13 months ago

#3653 closed change (fixed)

Modify application parameter in ABB for iOS

Reported by: sporz Assignee: fhd
Priority: P4 Milestone: Adblock-Browser-for-iOS-1.5.2
Module: Adblock-Browser-for-iOS Keywords:
Cc: mario, pavelz, Kirill, fhd Blocked By:
Blocking: Platform: Adblock Browser for iOS
Ready: yes Confidential: no
Tester: Scheer Verified working: yes
Review URL(s):

Description (last modified by mario)


We rely on application parameters sent when downloading resources like easylist.txt, etc. as basis for debugging and analyses.
AdblockBrowser for iOS currently provides application=chrome which cannot be linked to the product.

What to change

Change the query string parameter while requesting filter lists to application=adblockbrowserios

Change History (15)

comment:1 Changed 2 years ago by sporz

  • Cc Kirill added

comment:2 Changed 2 years ago by sporz

  • Cc pavelz added; pavel@… removed

comment:3 Changed 2 years ago by sporz

  • Sensitive unset

comment:4 Changed 2 years ago by fhd

  • Owner set to fhd
  • Priority changed from Unknown to P3
  • Ready set

comment:5 Changed 2 years ago by fhd

Here's a start:

This will only set application correctly since we have no mechanism for determining applicationVersion yet. But I suppose it'll be good enough for now?

If so, we need a follow-up issue for setting it properly. And we should think about platform and platformVersion while at it. Does it make more sense to have iOS or Kitt as the platform?

comment:6 Changed 2 years ago by sporz

From a data perspective, application now and applicationVersion later sounds much better than both later =D

Last edited 2 years ago by sporz (previous) (diff)

comment:7 Changed 2 years ago by pavelz

platform depends on what other values are you used to in there. Does it say Firefox or Android for your Android browser?

comment:8 Changed 2 years ago by sporz

On adblockbrowser for android it says platform=gecko, platformVersion=40.0.

comment:9 Changed 2 years ago by pavelz

That would interpolate to platform kitt. The fact that it's iOS is inseparable. Then there is a question of how you determine the iOS version, but you don't know that on Android either, do you? Are you even interested in the underlying OS version?

comment:10 Changed 2 years ago by pavelz

  • Cc fhd added

@fhd a proposal: you declare a property/setter on AdblockPlusApi in api.js and we set/call it for you with a version string as soon as the background script is loaded

comment:11 Changed 2 years ago by mario

  • Priority changed from P3 to P4

comment:12 Changed 20 months ago by pavelz

  • Keywords blocked added

comment:13 Changed 17 months ago by mario

  • Description modified (diff)
  • Keywords blocked removed
  • Resolution set to fixed
  • Status changed from new to closed
  • Summary changed from Modify application and applicationVersion parameters in ABB for iOS to Modify application parameter in ABB for iOS

I'm closing this issue, as the application parameter has already landed. Changed the description to reflect this scope. As suggested by fhd, however, I've filed follow-up issues to work on the remaining discussed parameters:

  • #4785: Actual change
  • #4786: Preparation of adblockplusadblockbrowserios to facilitate the changes of #4785

comment:14 Changed 16 months ago by mario

  • Milestone set to Adblock-Browser-for-iOS-next

comment:15 Changed 13 months ago by scheer

  • Tester changed from Unknown to Scheer
  • Verified working set
  • The query string parameter while requesting filter lists now shows as application=adblockbrowserios

Recent fetch shows:

GET /exceptionrules.txt?addonName=adblockplusadblockbrowserios&addonVersion= HTTP/1.1

ABB iOS 1.5.2 (Core 1.5.2)
iPhone 6S Plus
iOS 10.2

Note: See TracTickets for help on using tickets.