Bookmarks are not synced using a spanner account on iOS

[tublitzed]

Via Stefan on Bugzilla #1597192:

Build:
Version 20.1 (16561);

Devices:

iPhone SE (13.1.2);
iPhone 11 (13.1.2).

Prerequisites

You are connected with multiple devices(Desktop/Android/iOS) on the same spanner account.

Steps to reproduce:

Make some browsing on the Android device(bookmarks, logins, history);
Sync on the Android device;
Sync the iOS;

Expected results
All your browsing data are synced on the iOS device.

Actual results
The bookmarks data are not synced on iOS using the spanner account.

Notes:

History and logins are synced properly;
The sync doesn't work in both directions for FF on iOS;

[tublitzed]

Re-opening. Just confirmed this is still happening in at least one account in production (mine, sync id: 130387415) using firefox-ios 20.2. This is against Durable Sync 0.1.7

I'd guess that this is related to #342 which looks like an issue within the Rust components. That would explain why we're seeing some types of data sync (ie, History, which is a sync engine that is not yet wired to the Rust components) and not others.

[tublitzed]

FWIW, I'm going to test this one again later today after I get another test account (131750265) migrated over to spanner in prod. I've had some issues already with 130387415 so it's possible that what's happening there isn't happening to iOS across the board.

[tublitzed]

Confirmed that this is happening against newly created account as well.

[tublitzed]

@pjenvey just tracked this down, which looks like we're not allowing ENV var/server config to override maxTotalRecords, which might explain this (ie, we hit spanner mut limit).

[tublitzed]

Looks like this fix may have resolved this. Going to keep this one in review until we get QA signoff here in the related bugzilla issue.