-
Notifications
You must be signed in to change notification settings - Fork 152
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
Double bridging rule files #669
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM, pending matrix-org/matrix-appservice-bridge#85
rules.sample.yaml
Outdated
@@ -0,0 +1,10 @@ | |||
# A set of regexes to match against joined members in rooms. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should probably have a more descriptive name, like 'provisioning.rules.sample.yaml'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still LGTM
Depends on: matrix-org/matrix-appservice-bridge#85
This change allows you to do rule specifying on what is allowed to be provisioned to avoid some cases of double bridging. New provision requests would check against the rule file to see if the members in a certain room were excluded from being bridged e.g. not mixing "@snoonet-irc:matrix.org" and "@gimpnet-irc:matrix.org" in the same room.
This doesn't yet affect existing bridged rooms, which would need us to manually intervene.