Skip to content

/ ghostery-extension

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GH-2289: Add sub status to CMP fetch URL #694

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
+72 −56
Open

GH-2289: Add sub status to CMP fetch URL #694

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4419ef2
Add ss param to CMP URL build
leuryr Feb 26, 2021
ce987a9
Move query build to utils, factor out getSubStatus
leuryr Mar 8, 2021
851510b
Merge in develop
wlycdgr Mar 9, 2021
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Next
Loading status checks…

Add ss param to CMP URL build

  • Loading branch information
@leuryr
leuryr committed Feb 26, 2021
commit 4419ef2d0137adfa4fb67b7709bbe5854112f191
17 src/classes/CMP.js
View file
Open in desktop
@@ -87,6 +87,20 @@ class CMP {
}

static _buildUrl() {
let subStatus = 'free';
if (conf.account && conf.account.subscriptionData) {
switch (conf.account.subscriptionData.productName) {
case 'Ghostery Plus':
subStatus = 'plus';
break;
case 'Ghostery Premium':
subStatus = 'premium';
break;
default:
break;
}
}

return (`${CMP_BASE_URL}/check
?os=${encodeURIComponent(BROWSER_INFO.os)}
&hw=${encodeURIComponent(conf.enable_human_web ? '1' : '0')}
@@ -97,7 +111,8 @@ class CMP {
&ua=${encodeURIComponent(BROWSER_INFO.name)}
&lc=${encodeURIComponent(conf.last_cmp_date)}
&v=${encodeURIComponent(conf.cmp_version)}
&l=${encodeURIComponent(conf.language)}`
&l=${encodeURIComponent(conf.language)}
&ss=${encodeURIComponent(subStatus)}`
);
}

ProTip! Use n and p to navigate between commits in a pull request.