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

Spec @mentions #1067

Closed
ara4n opened this issue Nov 14, 2017 · 5 comments
Closed

Spec @mentions #1067

ara4n opened this issue Nov 14, 2017 · 5 comments
Assignees
Labels
kind:core MSC which is critical to the protocol's success merged A proposal whose PR has merged into the spec! proposal A matrix spec change proposal spec-omission implemented but not currently specified

Comments

@ara4n
Copy link
Member

ara4n commented Nov 14, 2017

Documentation: https://docs.google.com/document/d/1oRhw3DJhbVAKkHAEgyt6ccV82wtXR_11qY7JqMcesUU/edit
Author: @lukebarnard1
Date: 14/07/2017

I.e. spell out that hyperlinking to a matrix.to URL in an HTML is a namespacing operation to generate an @mention pill

@richvdh richvdh added the spec-omission implemented but not currently specified label Nov 14, 2017
@non-Jedi
Copy link
Contributor

non-Jedi commented Nov 15, 2017

Where is the google-doc for this? I don't understand how the mention pills relate to the spec. I'm also in general opposed to including matrix.to in any way in the spec.

@jevolk
Copy link
Contributor

jevolk commented Nov 15, 2017

@ara4n
Copy link
Member Author

ara4n commented Nov 15, 2017

@non-Jedi - I think you are misunderstanding how matrix.to is being used here. It is very much as an namespace (like an xmlns), so that clients which are aware of it can treat it like a URI scheme. Clients which are not aware of it will treat it like a dumb URI and fall through to the matrix.to webapp itself (which is provided by matrix.org for the benefit of the whole Matrix ecosystem) but it does not form a hard dependency on the matrix.to service whatsoever. it is just an arbitrary opaque namespace, similar to matrix:// or mx:// URLs or whatever, but one which happens for convenience to fall through to a real endpoint for stupid clients or users..

@non-Jedi
Copy link
Contributor

@ara4n I did understand; I just disagree with the decision, and I hope it
hasn't been made yet so that I might convince you of my point of view.

My contention is that no part of a protocol that's intentionally federated
should be linked to any singleton that can be controlled. While I realize
that it is being used as a namespace primarily, in its plain reading, a
matrix.to URL is a URL, and you admit that the reason matrix.to was chosen
was so that it could be used as a URL by "dumb" clients. While the URL is
controlled by an entity that I trust right now, there is no guarantee that
it will continue to be so in the future. Tbh, I'm not all that concerned
about an attacker controlling matrix.to; I'm concerned about matrix
maintaining the principle of federation: there are no privileged servers.
While matrix.to is not a matrix server, if it's included in the spec, it
will still be a privileged server.

I am completely fine with matrix.to being used as a de-facto namespace. I
strongly disagree with such behavior being part of the spec.

@non-Jedi
Copy link
Contributor

Thanks @jevolk. I had seen that doc before but somehow assumed it was a Riot design doc rather than a spec proposal.

@ara4n ara4n added spec-pr-missing Proposal has been implemented and is being used in the wild but hasn't yet been added to the spec proposal A matrix spec change proposal and removed proposal-passed-review labels May 14, 2018
turt2live added a commit to turt2live/matrix-doc that referenced this issue Aug 22, 2018
Implements the proposal over at matrix-org#1067

Includes some specification for how matrix.to is structured, and how it is intended to be replaced.
@turt2live turt2live self-assigned this Aug 22, 2018
turt2live added a commit to turt2live/matrix-doc that referenced this issue Aug 22, 2018
Implements the proposal over at matrix-org#1067

Includes some specification for how matrix.to is structured, and how it is intended to be replaced.
@richvdh richvdh added merged A proposal whose PR has merged into the spec! and removed spec-pr-missing Proposal has been implemented and is being used in the wild but hasn't yet been added to the spec labels Aug 30, 2018
@turt2live turt2live added the kind:core MSC which is critical to the protocol's success label Apr 21, 2020
RiotTranslateBot pushed a commit to RiotTranslateBot/matrix-doc that referenced this issue Aug 22, 2023
…atrix-org#1067)

* clarify federation Authorization header an add destination property

* add changelogs

* some clarifications

* more clarifications, fixes

* use HTML in the added-in/changed-in shortcodes

* Apply suggestions from code review

Co-authored-by: Travis Ralston <travpc@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:core MSC which is critical to the protocol's success merged A proposal whose PR has merged into the spec! proposal A matrix spec change proposal spec-omission implemented but not currently specified
Projects
None yet
Development

No branches or pull requests

6 participants