Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ElementR: Add CryptoApi.getCrossSigningStatus #3452

Merged
merged 18 commits into from
Jun 9, 2023

Conversation

florianduros
Copy link
Contributor

@florianduros florianduros commented Jun 6, 2023

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

Here's what your changelog entry will look like:

✨ Features

@florianduros florianduros changed the title Add crypto.getCrossSigningStatus ElementR: Add crypto.getCrossSigningStatus Jun 6, 2023
@florianduros florianduros changed the title ElementR: Add crypto.getCrossSigningStatus ElementR: Add CryptoApi.getCrossSigningStatus Jun 6, 2023
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good so far, a few suggestions!

src/@types/crypto.ts Outdated Show resolved Hide resolved
src/@types/crypto.ts Outdated Show resolved Hide resolved
src/crypto-api.ts Outdated Show resolved Hide resolved
src/crypto-api.ts Outdated Show resolved Hide resolved
src/rust-crypto/rust-crypto.ts Outdated Show resolved Hide resolved
spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
src/@types/crypto.ts Outdated Show resolved Hide resolved
spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
fetchMock.get("express:/_matrix/client/r0/user/:userId/account_data/:type", {});

// we expect a request to upload signatures for our device ...
fetchMock.post({ url: "path:/_matrix/client/v3/keys/signatures/upload", name: "upload-sigs" }, {});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to document these endpoint names, in the doc-comments for mockCrossSigningRequest. They form part of the interface of those functions, which the test is relying on.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to be clear, by "endpoint names", I mean things like upload-sigs.

spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
src/crypto-api.ts Outdated Show resolved Hide resolved
src/crypto-api.ts Outdated Show resolved Hide resolved
src/crypto-api.ts Outdated Show resolved Hide resolved
src/secret-storage.ts Outdated Show resolved Hide resolved
src/secret-storage.ts Outdated Show resolved Hide resolved
src/secret-storage.ts Outdated Show resolved Hide resolved
@florianduros
Copy link
Contributor Author

I'm adding new tests for the new secret-storage file.

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we're definitely nearly there! A few more things to fix.

spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
spec/integ/crypto/cross-signing.spec.ts Outdated Show resolved Hide resolved
src/rust-crypto/secret-storage.ts Outdated Show resolved Hide resolved
src/rust-crypto/secret-storage.ts Outdated Show resolved Hide resolved
src/rust-crypto/secret-storage.ts Outdated Show resolved Hide resolved
src/rust-crypto/secret-storage.ts Outdated Show resolved Hide resolved
src/rust-crypto/secret-storage.ts Outdated Show resolved Hide resolved
spec/unit/rust-crypto/secret-storage.spec.ts Outdated Show resolved Hide resolved
spec/unit/rust-crypto/secret-storage.spec.ts Outdated Show resolved Hide resolved
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@florianduros florianduros added this pull request to the merge queue Jun 9, 2023
Merged via the queue into develop with commit 2766146 Jun 9, 2023
@florianduros florianduros deleted the florianduros/element-r/add-cross-signing-status branch June 9, 2023 13:27
su-ex added a commit to SchildiChat/matrix-js-sdk that referenced this pull request Feb 22, 2024
* Introduce a new `Crypto.Verifier` interface, and deprecate direct access to `VerificationBase`, `SAS` and `ReciprocateQRCode` ([\matrix-org#3414](matrix-org#3414)).
* Add `rust-crypto#isCrossSigningReady` implementation ([\matrix-org#3462](matrix-org#3462)). Contributed by @florianduros.
* OIDC: Validate `m.authentication` configuration ([\matrix-org#3419](matrix-org#3419)). Contributed by @kerryarchibald.
* ElementR: Add `CryptoApi.getCrossSigningStatus` ([\matrix-org#3452](matrix-org#3452)). Contributed by @florianduros.
* Extend stats summary with call device and user count based on room state ([\matrix-org#3424](matrix-org#3424)). Contributed by @toger5.
* Update MSC3912 implementation to use `with_rel_type` instead of `with_relations` ([\matrix-org#3420](matrix-org#3420)).
* Export thread-related types from SDK ([\matrix-org#3447](matrix-org#3447)). Contributed by @stas-demydiuk.
* Use correct /v3 prefix for /refresh ([\matrix-org#3016](matrix-org#3016)). Contributed by @davidisaaclee.
* Fix thread list being ordered based on all updates ([\matrix-org#3458](matrix-org#3458)). Fixes element-hq/element-web#25522.
* Fix: handle `baseUrl` with trailing slash in `fetch.getUrl` ([\matrix-org#3455](matrix-org#3455)). Fixes element-hq/element-web#25526. Contributed by @kerryarchibald.
* use cli.canSupport to determine intentional mentions support ([\matrix-org#3445](matrix-org#3445)). Fixes element-hq/element-web#25497. Contributed by @kerryarchibald.
* Make sliding sync linearize processing of sync requests ([\matrix-org#3442](matrix-org#3442)).
* Fix edge cases around 2nd order relations and threads ([\matrix-org#3437](matrix-org#3437)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants