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

Implement Room Privacy Modes #100

Merged
merged 10 commits into from
Apr 6, 2021
Merged

Implement Room Privacy Modes #100

merged 10 commits into from
Apr 6, 2021

Conversation

cblgh
Copy link
Contributor

@cblgh cblgh commented Mar 24, 2021

This PR tracks the progress of implementing support for the SSB Rooms 2 concept of Privacy Modes.

The privacy modes control which types of users are allowed to perform certain actions, such as creating invites. In the open mode, any person may visit the room and create an invite to it. In the restricted mode, only existing room members with the roles of admin or moderator may invite new users.

Privacy modes:

  • Open
  • Community
  • Restricted

Closes #96

Outstanding todos:

  • Create an open invite view (on the basic description page index.html if mode is open?)
  • Fix tests to take / create mock config (after sqlite database?)

Copy link
Member

@cryptix cryptix left a comment

Choose a reason for hiding this comment

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

The idea and extension of struct's is correct..

Just some semantics of how it's supposed to work.

@@ -70,6 +74,23 @@ func (h *Handler) endpoints(_ context.Context, req *muxrpc.Request, snk *muxrpc.
return err
}

// cblgh:
// * reject if key in deny list / DeniedKeysService
Copy link
Member

Choose a reason for hiding this comment

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

the DeniedKeysService lookup needs to be added to roomsrv/init_network.go, not here.

I agree that endpoints is a confusion name, it's really just potentiall endpoints and tangentiall to the actual connection building.

Copy link
Member

Choose a reason for hiding this comment

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

#89 🐈

Copy link
Contributor Author

@cblgh cblgh Mar 26, 2021

Choose a reason for hiding this comment

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

yeah i know, this is just a note to use it + remember what the service was called

edit: oh wait, maybe i misunderstood actually, hm. i'll have a look inside roomsrv/init_network.go!

Copy link
Member

@cryptix cryptix left a comment

Choose a reason for hiding this comment

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

Just two small things.

@cblgh cblgh mentioned this pull request Apr 1, 2021
14 tasks
@staltz staltz linked an issue Apr 6, 2021 that may be closed by this pull request
@staltz
Copy link
Member

staltz commented Apr 6, 2021

@cblgh I also linked issue #12 with this PR

@cblgh cblgh force-pushed the privacy-modes branch 2 times, most recently from 502f130 to 377dc79 Compare April 6, 2021 14:15
@cryptix cryptix merged commit 77b1f52 into master Apr 6, 2021
@cryptix cryptix deleted the privacy-modes branch April 6, 2021 15:34
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.

Implementing privacy modes Database model for privacy modes
3 participants