1571033 - Don't reset the homepage before setting the default in policy

Status: RESOLVED FIXED

(Firefox :: Enterprise Policies, defect, P3)

Description

[Mike Kaply [:mkaply]]

Currently we reset the user's homepage in policy before setting the default.

This means users lose their homepage when the policy is set. We shouldn't do that.

Comment 1

[Mike Kaply [:mkaply]]

Attachment: Bug 1571033 - Don't clear user homepage when setting default via policy.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/398e8b80d4a8
Don't clear user homepage when setting default via policy. r=mconley

Comment 3

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/398e8b80d4a8

Comment 4

[Mike Kaply [:mkaply]]

Comment on attachment 9082677 [details]
Bug 1571033 - Don't clear user homepage when setting default via policy.

Beta/Release Uplift Approval Request

• User impact if declined: Homepage gets reset when the admin changes the default password (this is blocking firefox deployments)
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Policy only.
• String changes made/needed:

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Policy only
• User impact if declined: Homepage gets reset when the admin changes the default password (this is blocking firefox deployments)
• Fix Landed on Version: 70
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Policy only.
• String or UUID changes made by this patch:

Comment 5

[Julien Cristau [:jcristau]]

Comment on attachment 9082677 [details]
Bug 1571033 - Don't clear user homepage when setting default via policy.

fix for an enterprise policy, approved for 68.1esr and 69.0b16

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/releases/mozilla-esr68/rev/b392c57375cc

Comment 7

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/releases/mozilla-beta/rev/fa4ecfc3c662