1.160.0 - #65
Conversation
Annoying! Will lave that to upstream and we'll get updates as we rebase.
Signed-off-by: Sumner Evans <sumner.evans@automattic.com>
Still broken because the opentracing module upstream uses has long been retried and setuptools broke it.
- reduce hardcoded task parallelism - add hardcoded post task sleep
Co-authored-by: Codex <codex@openai.com>
|
|
|
| content: JsonDict, | ||
| expected_revision_id: str | None = None, | ||
| ) -> None: | ||
| content_json = json_encoder.encode(content) |
There was a problem hiding this comment.
Functional · Ignored bridge-bot filtering never reaches the stored account data blob
content_json is encoded here, before the bridge-bot filtering below rewrites content["ignored_users"]. The upsert at L896-902 stores this pre-filter content_json, so the account_data blob synced to clients via /sync still lists the bridge bots as ignored. Only the denormalized ignored_users table (built from the mutated content at L904+) is filtered, which is why test_ignoring_bot_users (asserting via store.ignored_users()) still passes. Result: the client sees bots as ignored while the server does not actually ignore them, defeating the intent of commit 0e0421a for the client-visible blob. Fix: move this encode to just before the _upsert_account_data_txn call, after the filtering block.
No description provided.