Skip to content
This repository has been archived by the owner on Feb 29, 2020. It is now read-only.

Linux repacks detected as unofficial builds #3769

Closed
Mardak opened this issue Oct 26, 2017 · 4 comments
Closed

Linux repacks detected as unofficial builds #3769

Mardak opened this issue Oct 26, 2017 · 4 comments

Comments

@Mardak
Copy link
Member

Mardak commented Oct 26, 2017

Also see https://bugzilla.mozilla.org/show_bug.cgi?id=1411867

const IS_UNOFFICIAL_BUILD = Services.prefs.getStringPref("app.update.channel") === "default";

The only pref using a custom _dev value right now:

["telemetry", {
title: "Enable system error and usage data collection",
value: true,
value_local_dev: false

So without a fix, we miss out on telemetry from these types of repacks.

@Mardak
Copy link
Member Author

Mardak commented Oct 30, 2017

It was suggested to use MOZILLA_OFFICIAL instead of the channel.
http://searchfox.org/mozilla-central/source/toolkit/modules/AppConstants.jsm#42

@Mardak Mardak added Bug and removed Regression labels Oct 31, 2017
@Mardak
Copy link
Member Author

Mardak commented Nov 9, 2017

So, MOZILLA_OFFICIAL won't be the same behavior that we have now, which I suppose is the purpose of this bug. @k88hudson, where do we want value_local_dev prefs to be set differently? Here's the differences in AppConstants among various builds:

local:

"NIGHTLY_BUILD": true,
"RELEASE_OR_BETA": false,
"MOZILLA_OFFICIAL": false,
"MOZ_OFFICIAL_BRANDING": false,
"MOZ_DEV_EDITION": false,
"MOZ_TELEMETRY_REPORTING": false,
"MOZ_TELEMETRY_ON_BY_DEFAULT": false,
"MOZ_REQUIRE_SIGNING": false,
"MOZ_ALLOW_LEGACY_EXTENSIONS": true,
"MOZ_UPDATE_CHANNEL": "default",
"SOURCE_REVISION_URL": "",

try:

"NIGHTLY_BUILD": true,
"RELEASE_OR_BETA": false,
"MOZILLA_OFFICIAL": true,
"MOZ_OFFICIAL_BRANDING": false,
"MOZ_DEV_EDITION": false,
"MOZ_TELEMETRY_REPORTING": false,
"MOZ_TELEMETRY_ON_BY_DEFAULT": false,
"MOZ_REQUIRE_SIGNING": false,
"MOZ_ALLOW_LEGACY_EXTENSIONS": true,
"MOZ_UPDATE_CHANNEL": "default",
"SOURCE_REVISION_URL": "https://hg.mozilla.org/try/rev/…",

autoland:

"NIGHTLY_BUILD": true,
"RELEASE_OR_BETA": false,
"MOZILLA_OFFICIAL": true,
"MOZ_OFFICIAL_BRANDING": false,
"MOZ_DEV_EDITION": false,
"MOZ_TELEMETRY_REPORTING": true,
"MOZ_TELEMETRY_ON_BY_DEFAULT": true,
"MOZ_REQUIRE_SIGNING": false,
"MOZ_ALLOW_LEGACY_EXTENSIONS": true,
"MOZ_UPDATE_CHANNEL": "default",
"SOURCE_REVISION_URL": "https://hg.mozilla.org/integration/autoland/rev/…",

nightly:

"NIGHTLY_BUILD": true,
"RELEASE_OR_BETA": false,
"MOZILLA_OFFICIAL": true,
"MOZ_OFFICIAL_BRANDING": false,
"MOZ_DEV_EDITION": false,
"MOZ_TELEMETRY_REPORTING": true,
"MOZ_TELEMETRY_ON_BY_DEFAULT": true,
"MOZ_REQUIRE_SIGNING": false,
"MOZ_ALLOW_LEGACY_EXTENSIONS": true,
"MOZ_UPDATE_CHANNEL": "nightly",
"SOURCE_REVISION_URL": "https://hg.mozilla.org/mozilla-central/rev/…",

aurora:

"NIGHTLY_BUILD": false,
"RELEASE_OR_BETA": true,
"MOZILLA_OFFICIAL": true,
"MOZ_OFFICIAL_BRANDING": false,
"MOZ_DEV_EDITION": true,
"MOZ_TELEMETRY_REPORTING": true,
"MOZ_TELEMETRY_ON_BY_DEFAULT": false,
"MOZ_REQUIRE_SIGNING": false,
"MOZ_ALLOW_LEGACY_EXTENSIONS": true,
"MOZ_UPDATE_CHANNEL": "aurora",
"SOURCE_REVISION_URL": "https://hg.mozilla.org/releases/mozilla-beta/rev/…",

beta:

"NIGHTLY_BUILD": false,
"RELEASE_OR_BETA": true,
"MOZILLA_OFFICIAL": true,
"MOZ_OFFICIAL_BRANDING": true,
"MOZ_DEV_EDITION": false,
"MOZ_TELEMETRY_REPORTING": true,
"MOZ_TELEMETRY_ON_BY_DEFAULT": false,
"E10S_TESTING_ONLY": false,
"MOZ_REQUIRE_SIGNING": true,
"MOZ_ALLOW_LEGACY_EXTENSIONS": false,
"MOZ_UPDATE_CHANNEL": "beta",
"SOURCE_REVISION_URL": "https://hg.mozilla.org/releases/mozilla-beta/rev/…",

release:

"NIGHTLY_BUILD": false,
"RELEASE_OR_BETA": true,
"MOZILLA_OFFICIAL": true,
"MOZ_OFFICIAL_BRANDING": true,
"MOZ_DEV_EDITION": false,
"MOZ_TELEMETRY_REPORTING": true,
"MOZ_TELEMETRY_ON_BY_DEFAULT": false,
"E10S_TESTING_ONLY": false,
"MOZ_REQUIRE_SIGNING": true,
"MOZ_ALLOW_LEGACY_EXTENSIONS": false,
"MOZ_UPDATE_CHANNEL": "release",
"SOURCE_REVISION_URL": "https://hg.mozilla.org/releases/mozilla-release/rev/…",

@tspurway tspurway removed the fx58 label Nov 13, 2017
@Mardak Mardak added this to the Ocean's Eleven (January 7) milestone Nov 13, 2017
@Mardak Mardak added the fx59 label Nov 13, 2017
@Mardak Mardak self-assigned this Nov 14, 2017
@Mardak
Copy link
Member Author

Mardak commented Nov 14, 2017

Got confirmation that we just care about actually local developer builds getting special prefs.

@Mardak
Copy link
Member Author

Mardak commented Dec 2, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants