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

Fix support for password protected rooms #1286

Open
robertfoss opened this issue Apr 9, 2021 · 3 comments
Open

Fix support for password protected rooms #1286

robertfoss opened this issue Apr 9, 2021 · 3 comments
Labels
needs-testing S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.

Comments

@robertfoss
Copy link

robertfoss commented Apr 9, 2021

Describe the bug

There are two somewhat separate issues that I've found

  • There is no (documented?) way of joining password protected irc channels through the Explore Public Rooms menu
    • This can be worked around by issuing a !join #channel password to @appservice-irc:matrix.org
  • However this password is not remembered when the room is reconnected to (after an irc server restart for example), so this room will fail to connect and will be removed from the rooms list silently without notifying the user.

To Reproduce

  • Manually join password protected irc channel by sending !join #channel password to @appservice-irc:matrix.org
  • Wait for irc server to restart
  • After restart is completed only non-password protected irc channels will be rejoined and listed in your rooms view.

Expected behavior

I would expect the password associated with a channel to be stored wherever my irc channel membership list is stored, or possibly where the irc server password is stored.

For the web app:

  • Browser: Chrome, Firefox, Safari, Edge? which version?
  • OS: Windows, macOS, Ubuntu, Arch Linux, etc?
  • URL: matrix.org

For the desktop app:

  • OS: Ubuntu 20.10
  • Version: Element version: 1.7.23
@jaller94
Copy link
Contributor

Presumably fixed by element-hq/element-web#16829

Untested.

@jaller94 jaller94 added T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems. S-Major Severely degrades major functionality or product features, with no satisfactory workaround needs-testing labels May 16, 2022
@robertfoss
Copy link
Author

This issue still exists. Channel passwords are not stored anywhere.

Could not join #linaro-lt-qcom on '[irc.oftc.net](http://irc.oftc.net/)': err_badchannelkey

@Half-Shot
Copy link
Contributor

I don't think this is fixed. The upstream issue was redirected here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-testing S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.
Projects
None yet
Development

No branches or pull requests

3 participants