Skip to content

BUILD: Move owid-js to the key selection fix at 9aeb0d24 - #190

Merged
Automation51D merged 1 commit into
mainfrom
fix/owid-key-selection-by-period-start
Sep 6, 2026
Merged

BUILD: Move owid-js to the key selection fix at 9aeb0d24#190
Automation51D merged 1 commit into
mainfrom
fix/owid-key-selection-by-period-start

Conversation

@jwrosewell

Copy link
Copy Markdown
Contributor

Summary

Moves the owid dependency of fiftyone.pipeline.did from github:51Degrees/owid-js#8363541e to github:51Degrees/owid-js#9aeb0d24, the head of the 51Degrees fork after it took SWAN-community/owid-js main at 9cf95fea. The three lines that name the commit change (package.json, remote_package.json and the readme sentence that explains the reference). package-lock.json is not tracked in this repository.

Why

On 4 September 2026 the OWID ports were found to select a creator's signing key by the moment the key was generated rather than by the start of its period. The 51Degrees cloud writes thirteen weeks of keys in one run, so an identifier dated 4 September selected the key starting 7 September and genuine identifiers were reported as forged. The rule was corrected in the OWID specification (SWAN-community/owid#7) and in every port on 5 September 2026. The JavaScript fix is SWAN-community/owid-js#11, which also makes the creator request name the version the OWID carries (a version 2 OWID asks the /owid/api/v2/ end point), and adds a test against the real published schedule and a real identifier.

FodId.verify() and FodId.checkSignature() in this package verify offline against a key the caller supplies, so their behaviour does not change. The bump keeps the bundled OWID source (see bundleDependencies) identical to what the other 51Did clients compile in, and picks up the version segment fix for anyone who uses the OWID network path directly.

Verification

  • npm ci in fiftyone.pipeline.did, then npx jest: 139 passed, 2 skipped (the live tests), 3 of 4 suites run (1 skipped).
  • The installed node_modules/owid carries the new owid.published-key.test.js and testdata/, and v1.js builds the creator path from the OWID's own version byte.
  • Cross-port fixture check: the real identifier the cloud issued on 4 September 2026 with the real thirty key schedule from id/key verifies under the key starting 31 August 2026, and the same schedule with every start moved a week later answers Invalid.

Notes

  • refactor: switch to npm workspaces #174 (npm workspaces) also edits fiftyone.pipeline.did/package.json and remote_package.json. Whichever of the two merges second needs a one line rebase.
  • Nothing else in this repository references owid-js.

Produced with AI assistance and checked against the outputs above.

The 51Degrees/owid-js fork took SWAN-community/owid-js#11 on
5 September 2026, which selects the creator's signing key by the start
of its period rather than the moment it was generated, and asks the
creator end point for the OWID's own version. package.json,
remote_package.json and the readme sentence that explains the reference
now name 9aeb0d24. FodId verifies offline against a supplied key, so its
behaviour is unchanged; the bundled OWID source now matches the other
51Did clients.

Jest: 139 passed, 2 skipped. Fixture check: the real 4 September
identifier verifies under the key starting 31 August 2026 and answers
Invalid on the schedule shifted a week.
@Automation51D
Automation51D merged commit 432799b into main Sep 6, 2026
1 check passed
@Automation51D
Automation51D deleted the fix/owid-key-selection-by-period-start branch September 6, 2026 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants