Skip to content

Update Matrix message send endpoint#4598

Open
tulir wants to merge 4 commits intositespeedio:mainfrom
tulir:patch-1
Open

Update Matrix message send endpoint#4598
tulir wants to merge 4 commits intositespeedio:mainfrom
tulir:patch-1

Conversation

@tulir
Copy link
Copy Markdown

@tulir tulir commented Mar 13, 2026

Switched:

  • Deprecated access_token query param -> Authorization header
  • Undocumented/non-standard POST endpoint -> standard PUT with a transaction ID
  • Old /r0 -> new /v3

References:

Your checklist for a pull request to sitespeed.io

Please review the guidelines for contributing to this repository.

  • I'm making a big change or adding functionality so I've already opened an issue proposing the change to other contributors, so I got feedback on the idea before took the time to write precious code
  • Check that your change/fix has corresponding unit tests (if applicable)
  • Squash commits so it looks sane
  • Update the documentation https://github.com/sitespeedio/sitespeed.io/tree/main/docs in another PR
  • Verify that the test works by running npm test and test linting by npm run lint

Description

Please describe your pull request and tell us the fix #

Thank you for making sitespeed.io even better!

Switched:

* Deprecated `access_token` query param -> `Authorization` header
* Undocumented/non-standard POST endpoint -> standard PUT with a transaction ID
* Old /r0 -> new /v3

References:
* matrix-org/matrix-spec-proposals#4127
* https://spec.matrix.org/v1.17/client-server-api/#put_matrixclientv3roomsroomidsendeventtypetxnid
Comment thread lib/plugins/matrix/send.js Outdated
Comment thread lib/plugins/matrix/send.js Outdated
Comment thread lib/plugins/matrix/send.js Outdated
@soulgalore
Copy link
Copy Markdown
Member

Thank you @tulir let me have look. Seems also to be many unrelated errors in the Actions, let me fix that in another branch before we update this.

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.

2 participants