Closed Bug 1739525 Opened 3 years ago Closed 3 years ago

`feature.enabled` is still in use even when server already removed it

Categories

(Firefox :: Nimbus Desktop Client, defect)

defect

Tracking

()

RESOLVED INVALID

People

(Reporter: Mardak, Unassigned)

References

Details

Similar to bug 1739007 with functionality broken with the removal of feature.enabled, there's some other code still relying on it such as ASRouter exposure ping:

https://searchfox.org/mozilla-central/rev/a12c2c2e59c92d8f969d8f3f290ab16919449c9d/browser/components/newtab/lib/ASRouter.jsm#344-352

There may be other existing uses of the direct object property access instead of using the isEnabled() wrapper. But even with isEnabled(), that assumes correct experiment feature value configuration with setting {enabled: true}

Actually, without feature.enabled I believe even treatment branch that set it to true will be treated as disabled because server no longer sets that value. This affects https://experimenter.services.mozilla.com/nimbus/emotive-pin-firefox-modal/details as the experiment hasn't ended enrollment, but doing so will remove feature.enabled from remote settings record breaking all the treatment branches.

Bug 1739666 might specifically fix the issue from comment 0, so this bug might be a dupe… unless there are still other usages of feature.enabled elsewhere.

Blocks: 1728989
Depends on: 1739666

Server is reverting the feature.enabled removal with https://github.com/mozilla/experimenter/pull/6631

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.