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

Split up, rename, and move ISecretStorageKeyInfo #3242

Merged
merged 2 commits into from
Apr 3, 2023

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Mar 31, 2023

This interface currently:

  • is in the wrong place (it is tied to the Crypto api, which is unnecessary)
  • assumes that we will be using an AES key, which is not necessarily the case
  • Has a bad name starting with I

This PR attempts to solve all three problems, by splitting the class up, renaming it, and moving it.

The old name is still exported for backwards compatibility.

This is the first step part of a larger piece of work in refactoring the SecretStorage interface (element-hq/element-web#24982)


This change is marked as an internal change (Task), so will not be included in the changelog.

@richvdh richvdh added the T-Task Tasks for the team like planning label Mar 31, 2023
@richvdh richvdh force-pushed the rav/refactor_secret_storage/cleanup_descriptors branch from d1f27ec to d711ed4 Compare March 31, 2023 13:47
@richvdh richvdh added this pull request to the merge queue Apr 3, 2023
Merged via the queue into develop with commit 41d3ffd Apr 3, 2023
@richvdh richvdh deleted the rav/refactor_secret_storage/cleanup_descriptors branch April 3, 2023 10:46
weeman1337 pushed a commit that referenced this pull request Apr 4, 2023
* Move SecretStorageKeyInfo interfaces out to a new module

* Replace usages of ISecretStorageKeyInfo with SecretStorageKeyDescription
weeman1337 added a commit that referenced this pull request Apr 6, 2023
* Retry processing potential poll events after decryption

* Point `typedoc` at `matrix.ts`, not `index.ts` (#3239)

This gets rid of the rather pointless "default" module in the generated docs.

* Split up, rename, and move `ISecretStorageKeyInfo` (#3242)

* Move SecretStorageKeyInfo interfaces out to a new module

* Replace usages of ISecretStorageKeyInfo with SecretStorageKeyDescription

* Skip clear text non-poll events

---------

Co-authored-by: Richard van der Hoff <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants