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

chore: Forbid local user from inviting remote user to a public room #59

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

jason-famedly
Copy link
Contributor

@jason-famedly jason-famedly commented Mar 6, 2025

requires #61
bullet point 2 of famedly/product-management#2925

Public rooms do not require an invite to join, but placing such an invite can circumvent the required blocking. By not allowing such an invite to be created, this should prevent that circumvention

@jason-famedly jason-famedly force-pushed the jason/prevent-pro-joining-remote-public-rooms branch 2 times, most recently from 6587e9b to ab1f672 Compare March 11, 2025 16:46
@jason-famedly jason-famedly force-pushed the jason/block-out-invites-to-local-pub-room branch from ff30ea5 to 9fc35e4 Compare March 11, 2025 18:46
@jason-famedly jason-famedly changed the base branch from jason/prevent-pro-joining-remote-public-rooms to jason/no-joining-remote-pub-rooms-with-invite March 11, 2025 18:49
@jason-famedly jason-famedly force-pushed the jason/no-joining-remote-pub-rooms-with-invite branch 2 times, most recently from 055ab1f to ac95424 Compare March 13, 2025 15:05
@jason-famedly jason-famedly force-pushed the jason/block-out-invites-to-local-pub-room branch from 9fc35e4 to 7c6db75 Compare March 13, 2025 16:12
@jason-famedly jason-famedly marked this pull request as ready for review March 13, 2025 16:16
@jason-famedly jason-famedly requested review from a team as code owners March 13, 2025 16:16
@jason-famedly jason-famedly force-pushed the jason/no-joining-remote-pub-rooms-with-invite branch from 3410f38 to 0301c42 Compare March 13, 2025 16:19
Copy link
Member

@nico-famedly nico-famedly left a comment

Choose a reason for hiding this comment

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

Do we have no tests, that test inviting into a public room after creation? Could you add a short one for that?

Base automatically changed from jason/no-joining-remote-pub-rooms-with-invite to main March 13, 2025 16:28
@jason-famedly jason-famedly force-pushed the jason/block-out-invites-to-local-pub-room branch from 7c6db75 to 0301c42 Compare March 13, 2025 16:30
@jason-famedly
Copy link
Contributor Author

um, what happened?

@jason-famedly
Copy link
Contributor Author

not sure why this closed. I didn't hit that button

@jason-famedly
Copy link
Contributor Author

open

@jason-famedly jason-famedly reopened this Mar 13, 2025
Copy link

codecov bot commented Mar 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.94%. Comparing base (0301c42) to head (409764c).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   97.92%   97.94%   +0.01%     
==========================================
  Files           9        9              
  Lines         772      778       +6     
  Branches      106      109       +3     
==========================================
+ Hits          756      762       +6     
  Misses         16       16              
Files with missing lines Coverage Δ
synapse_invite_checker/invite_checker.py 96.94% <100.00%> (+0.04%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0301c42...409764c. Read the comment docs.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jason-famedly jason-famedly force-pushed the jason/block-out-invites-to-local-pub-room branch from c0236b3 to d26c8b7 Compare March 13, 2025 17:05
@jason-famedly
Copy link
Contributor Author

Do we have no tests, that test inviting into a public room after creation? Could you add a short one for that?

c0236b3
There were, just got mixed up somewhere. Its a partial unblock of one of the newer ones

@jason-famedly jason-famedly force-pushed the jason/block-out-invites-to-local-pub-room branch from d26c8b7 to 409764c Compare March 13, 2025 17:16
@jason-famedly jason-famedly merged commit 409764c into main Mar 13, 2025
6 checks passed
@jason-famedly jason-famedly deleted the jason/block-out-invites-to-local-pub-room branch March 13, 2025 17:37
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