Skip to content

Conversation

@andybalaam
Copy link
Member


private cryptoBackend?: CryptoBackend; // one of crypto or rustCrypto
private readonly enableEncryptedStateEvents: boolean;
public enableEncryptedStateEvents: boolean;
Copy link
Contributor

Choose a reason for hiding this comment

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

If it's public (also for private), should we have tsdoc?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, added in f24d11a

Copy link
Member

Choose a reason for hiding this comment

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

It should explain what changing it does. Will changing it apply or do you need to restart the client. Will it retroactively decrypt state events?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed in 10936f2

…ublic

Improve the description of enableEncryptedStateEvents
@andybalaam andybalaam added this pull request to the merge queue Dec 18, 2025
Merged via the queue into develop with commit 97fcdb2 Dec 18, 2025
32 checks passed
@andybalaam andybalaam deleted the andybalaam/msc4362-encrypted-state branch December 18, 2025 14:17
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.

4 participants