Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Add security customisation points #5327

Merged
merged 1 commit into from
Oct 16, 2020
Merged

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented Oct 15, 2020

This adds various customisations point in the app for security related
decisions. By default, these do nothing, but would be customised at the
app level via module replacement (so that no changes are needed here in the
SDK).

See also element-hq/element-web#15475 for usage documentation.

Fixes element-hq/element-web#15350

@jryans jryans requested a review from a team October 15, 2020 16:37
jryans added a commit to element-hq/element-web that referenced this pull request Oct 15, 2020
matrix-org/matrix-react-sdk#5327 adds a concept of
"customisation points" to keep custom functionality contained. This adds some
general guidance on how to use them.
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

I think the type failures are a false alarm on this - looks good to me.

src/SecurityManager.ts Outdated Show resolved Hide resolved
This adds various customisations point in the app for security related
decisions. By default, these do nothing, but would be customised at the
app level via module replacement (so that no changes are needed here in the
SDK).

Fixes element-hq/element-web#15350
@jryans jryans force-pushed the jryans/sso-4s-integration branch from fb0a30d to 388cb0e Compare October 16, 2020 10:21
@jryans jryans merged commit ce8a4c0 into develop Oct 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add security customisation points
2 participants