New issue
Advanced search Search tips

Issue 850225 link

Starred by 10 users

Issue metadata

Status: ExternalDependency
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug



Sign in to add a comment

Mojave: Review importers from other browsers

Project Member Reported by rsesek@chromium.org, Jun 6 2018

Issue description

In macOS 10.14 Mojave, the system restricts access to sensitive data directories. This now includes the Safari browsing history and HTTP cookies. Apps that try to access these locations may be denied or crash (at time of writing, I have not tested Chrome's behavior).

Chrome tries to import from Safari by reading its databases directly (https://cs.chromium.org/chromium/src/chrome/utility/importer/safari_importer.mm?rcl=770f56faf8cd4a892c2d3d516434c62e40527d4a), which almost certainly is a violation of this restriction.

We will need to figure out if and how we handle importing going forward.

More details:
https://developer.apple.com/videos/play/wwdc2018/702/
 
Labels: Target-69
Owner: a...@chromium.org
Status: Assigned (was: Untriaged)
Attempting to import from Safari does nothing on 10.14 and spits an error to stderr: "sqlite error 14, errno -2: unable to open database file, sql: -- sqlite3_open()".

Comment 2 Deleted

Gentle ping to get an update on this issue as it is marked as RBS.
Thanks..!
Status: ExternalDependency (was: Assigned)
Blocked on communication with Apple.
M69 Stable promotion is coming VERY soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. Thank you.
avi@, any progress since your last comment #4 on August 1st as this is marked as M69 stable blocker which is coming soon.
No progress. Apple stays silent until they decide to speak. I can't control when they'll get back to me :(
M69 Stable promotion is coming VERY soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge into the release branch ASAP. Thank you.


Labels: -ReleaseBlock-Stable
This isn't a RB. We have no control over this.

Sign in to add a comment