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

[WIP] MSC3348: Server-Server Room based extensible communication #3348

Closed
wants to merge 11 commits into from
Closed

Conversation

FSG-Cat
Copy link
Contributor

@FSG-Cat FSG-Cat commented Aug 18, 2021

Rendered

Room for the discussion of this MSC exists at #CatMSC3348:feline.support

Signed-off-by: Catalan Lover catalanlover@protonmail.com

@FSG-Cat FSG-Cat changed the title [WIP] MSCXXXX: Server-Server Room based extensible communication [WIP] MSC3348: Server-Server Room based extensible communication Aug 18, 2021
…ion.md to 3348 Server-Server Room based extensible communication.md
Copy link
Member

@uhoreg uhoreg left a comment

Choose a reason for hiding this comment

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

Some spelling/grammar/formatting nit picking

@uhoreg uhoreg added kind:feature MSC for not-core and not-maintenance stuff proposal A matrix spec change proposal s2s Server-to-Server API (federation) labels Aug 18, 2021
@turt2live
Copy link
Member

@FSG-Cat we'll also need sign-off on this before it's eligible for FCP. The easiest way to do this is to leave a comment or add to the PR description the following:

Signed-off-by: Your Name <email@example.org>

FSG-Cat and others added 8 commits August 18, 2021 21:45
…on.md

Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
…on.md

Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
…on.md

Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
…on.md

Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
…on.md

Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
Removed a non needed Ack step since the Ack is in response to the accept and that accept can be treated as an ack. This brings the Round Trips needed down to 3.
With the negotiation mechanism rework sus2s upgrades only needs 2 events being sent before the Upgrade process can start with the third event being able to be sent at the same time as the upgrade process is initated.

### The identifiers that servers should use to identify the server users

The IDs the homeservers should use is currently recommended to be a simple @:identifier

Choose a reason for hiding this comment

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

This syntax is not clear. Does it mean a literal @: followed by an identifier or a literal @ followed by an identifier?

I see there are references later that seem to imply that it is the former but maybe an example right next to this would be helpful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The complete mxid under this proposal for the homeserver would be @:example.com in the case of example.com.

@turt2live turt2live added the needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. label Sep 28, 2021
@FSG-Cat
Copy link
Contributor Author

FSG-Cat commented Feb 6, 2022

Closing this since it wont progress under Cat.

@FSG-Cat FSG-Cat closed this Feb 6, 2022
@turt2live turt2live added the abandoned A proposal where the author/shepherd is not responsive label Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abandoned A proposal where the author/shepherd is not responsive kind:feature MSC for not-core and not-maintenance stuff needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. proposal A matrix spec change proposal s2s Server-to-Server API (federation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants