Closed Bug 1825874 Opened 1 year ago Closed 1 year ago

Ask the user to import the Safari passwords .csv file if they've chosen to import passwords from Safari

Categories

(Firefox :: Migration, task, P2)

task

Tracking

()

RESOLVED FIXED
115 Branch
Tracking Status
firefox115 --- fixed

People

(Reporter: mconley, Assigned: mconley)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Safari doesn't let us get at the entire collection of passwords in the keyring unless the user provides permission...for...each...password, which is a pretty garbage import experience.

Our current solution is actually to have the user import the CSV file that Safari can export in about:logins.

What we'd like to do is expose Passwords as a resource that is importable from Safari, but if the user chooses it, to navigate the user to the SAFARI_PASSWORD_PERMISSION page that was added in bug 1818789, and prompt the user to browse to the CSV file that Safari exports.

Then we can import the other Safari resources that we do have access to and the passwords all in one shot.

This depends on bug 1821187 because we're probably going to want to re-use the file migration mechanism for the special Safari case here.

Severity: -- → N/A
Priority: -- → P2

Note that the data review for the browser.migration.safari_password_file telemetry event is covered by the data-review in bug 1824786.

Safari doesn't let us import all passwords from Keychain at once. Instead,
the user needs to authorize each read from Keychain one by one. This is
not tenable for password import.

Instead, we show the user instructions on how they can export their passwords
from Safari into a CSV file that can then be imported. This patch adds support
for showing those instructions (which had landed earlier in bug 1818789) and
then importing from that CSV file upon selection.

The data-review for the safari_password_file Telemetry event being used here
was done earlier in bug 1824786.

Assignee: nobody → mconley
Status: NEW → ASSIGNED
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/863fae78db03
Offer to import passwords from a CSV file for Safari. r=kpatenio
Backout by abutkovits@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79fb1520edea
Backed out changeset 863fae78db03 for causing failures at browser_disabled_migrator.js. CLOSED TREE
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eb8042169586
Offer to import passwords from a CSV file for Safari. r=kpatenio
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
Flags: needinfo?(mconley)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: