MSC4213: Remove server_name parameter from join and knock endpoints
#4213
Conversation
Johennes
force-pushed
the
johannes/remove-server-name
branch
2 times, most recently
from
b894e89 to
a083ae4
Compare
Johennes
changed the title
Signed-off-by: Johannes Marbach <[email protected]>
Johennes
force-pushed
the
johannes/remove-server-name
branch
from
a083ae4 to
f011a86
Compare
turt2live
added
proposal
There was a problem hiding this comment.
Implementation requirements:
I'm tempted to mirror #4127 (comment) and say "none", but am second-guessing that MSC's requirements too. It might be beneficial to have evidence that developers are picking up the newer approaches before suggesting removal from the spec.
Thoughts very welcome.
There was a problem hiding this comment.
It might not count because they were all created by me but there are five implementations linked in the proposal's body, four of which have already landed.
|
Using the FCP mechanism to encourage comment/explicit blockers: @mscbot fcp merge |
|
Team member @mscbot has proposed to merge this. The next step is review by the rest of the tagged people: Concerns:
Once at least 75% of reviewers approve (and there are no outstanding concerns), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for information about what commands tagged team members can give me. |
mscbot
added
proposed-final-comment-period
turt2live
added
implementation-needs-checking
|
@mscbot concern implementation-needs-checking label |
mscbot
added
the
unresolved-concerns
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
|
|
||
| - synapse: https://github.com/element-hq/synapse/pull/17650 | ||
| - dendrite: https://github.com/matrix-org/dendrite/pull/3438 | ||
| - matrix-js-sdk: https://github.com/matrix-org/matrix-js-sdk/pull/4381 |
There was a problem hiding this comment.
It's possible I'm confused but matrix-org/matrix-js-sdk#4381 removes the unstable prefix from the 'via' param. This MSC is about removing the old server_name param, which is a different thing.
There was a problem hiding this comment.
The linked implementations are for MSC4156 which added the via param. They're listed to show that the old server_name parameter is safe to remove
There was a problem hiding this comment.
Yes, exactly. I think the server_name parameter will probably stay in implementations for quite a while unless they consciously choose to only support Matrix >= 1.12. But we don't need it in the spec anymore.
There was a problem hiding this comment.
Oh right, I see - I guess I was confused by your comment. In that case I guess I can vouch for the js-sdk impl, in that it now sends the via param.
There was a problem hiding this comment.
The server impl looks plausible too. I think we could remove the implementation-needs-checking label.?
richvdh
changed the title
join and knock endpoints
|
@mscbot resolve implementation-needs-checking label |
mscbot
removed
the
unresolved-concerns
turt2live
added
unresolved-concerns
Rendered
In line with matrix-org/matrix-spec#1700, the following disclosure applies:
I am a Systems Architect at gematik, Software Engineer at Unomed, Matrix community member and former Element employee. This proposal was written and published with my community member hat on.
FCP tickyboxes