Skip to content

Add temporary deprecation warning to README.md - #207

Merged
8go merged 1 commit into
8go:masterfrom
Benjamin-Loison:master
Jun 8, 2026
Merged

Add temporary deprecation warning to README.md#207
8go merged 1 commit into
8go:masterfrom
Benjamin-Loison:master

Conversation

@Benjamin-Loison

@Benjamin-Loison Benjamin-Loison commented Dec 1, 2025

Copy link
Copy Markdown
Collaborator

I don't want to be rude, and I don't enjoy having to open such a pull request to such a useful repository, but it seems necessary, see the proposed changes.

Related to Benjamin_Loison/matrix-nio/issues/2.

@Benjamin-Loison

Copy link
Copy Markdown
Collaborator Author

As far as I understand emitting a security warning notification would make sense, see Benjamin-Loison/matrix-commander/issues/39.

@bjornfor

Copy link
Copy Markdown

Ping @8go?

matrix-commander is effectively disabled in nixpkgs due to the security issue with olm. I think it's important to get this warning into the README, or else it sends a strong "this is a dead project" signal. Which would be unfortunate, as it seems to be the most recommended matrix CLI.

@8go
8go merged commit 86aefdf into 8go:master Jun 8, 2026
1 check failed
@8go

8go commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Sorry for the long delay, I apologize.

@Benjamin-Loison

Benjamin-Loison commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator Author

@8go It seems that I do not have access to emitting a security advisory, can you do it?

Screenshots showing that I do not have access: image image image image image image image

BTKBot

image image

Illustrates Improve_websites_thanks_to_open_source/issues/3662.

Related to Improve_websites_thanks_to_open_source/issues/3664.

I enabled on my end notifications to matrix-nio/matrix-nio/pull/555, 8go/matrix-commander and 8go/matrix-commander-rs.

@8go

8go commented Jun 8, 2026

Copy link
Copy Markdown
Owner

I added this:

GHSA-wchh-9x6h-7f6p

It is fully published now.

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.

3 participants