Skip to content

Skip processing policy server events through policy server#18605

Merged
anoadragon453 merged 3 commits intodevelopfrom
travis/ps-self-ignore
Jun 30, 2025
Merged

Skip processing policy server events through policy server#18605
anoadragon453 merged 3 commits intodevelopfrom
travis/ps-self-ignore

Conversation

@turt2live
Copy link
Copy Markdown
Member

This is to (primarily) allow a room to disable the policy server while it is down.

See https://github.com/matrix-org/matrix-spec-proposals/pull/4284/files#r2070860418 for related thoughts.

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Code style is correct (run the linters)

This is to (primarily) allow a room to disable the policy server while it is down.
@turt2live turt2live marked this pull request as ready for review June 28, 2025 00:03
@turt2live turt2live requested a review from a team as a code owner June 28, 2025 00:03
@anoadragon453 anoadragon453 enabled auto-merge (squash) June 30, 2025 10:06
@anoadragon453 anoadragon453 disabled auto-merge June 30, 2025 10:44
@anoadragon453
Copy link
Copy Markdown
Member

CI failures are due to flakes, merging manually.

@anoadragon453 anoadragon453 merged commit b35c648 into develop Jun 30, 2025
41 of 43 checks passed
@anoadragon453 anoadragon453 deleted the travis/ps-self-ignore branch June 30, 2025 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants