matrix-org / matrix-spec-proposals

Proposals for changes to the matrix specification
Apache License 2.0
982 stars 374 forks source link

MSC4156: Migrate server_name to via #4156

Closed Johennes closed 3 weeks ago

Johennes commented 2 months ago

Rendered

Fixes: https://github.com/matrix-org/matrix-spec/issues/884


In line with https://github.com/matrix-org/matrix-spec/issues/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

clokep commented 1 month ago

I think this is the reasonable way forward.

@mscbot fcp merge

mscbot commented 1 month ago

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.

turt2live commented 1 month ago

@mscbot concern Security considerations needs completing @mscbot concern General clarity @mscbot concern May require more client implementation (needs debate)

turt2live commented 1 month ago

Thanks for the updates :)

@mscbot resolve Security considerations needs completing @mscbot resolve General clarity

It appears as though the implementation thread isn't receiving replies, but the MSC is getting quite a bit of traffic, so:

@mscbot resolve May require more client implementation (needs debate)

mscbot commented 1 month ago

:bell: This is now entering its final comment period, as per the review above. :bell:

mscbot commented 3 weeks ago

The final comment period, with a disposition to merge, as per the review above, is now complete.

Johennes commented 3 weeks ago

Spec PR: https://github.com/matrix-org/matrix-spec/pull/1933

turt2live commented 2 weeks ago

Merged 🎉