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

When "Publish this room to the public in matrix.org's room directory?" fails it should display why #27981

Closed
ell1e opened this issue Aug 30, 2024 · 7 comments · Fixed by matrix-org/matrix-react-sdk#12941
Labels
A-Error-Message A-Room-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ell1e
Copy link

ell1e commented Aug 30, 2024

Steps to reproduce

  1. Open up any room you administrate on matrix.org
  2. Open "Settings" > "General"
  3. Make sure your room has an alias with a name set
  4. With "Publish this room to the public in matrix.org's room directory?" previously toggled off, toggle it on

Outcome

What did you expect?

It either stays turned on or there's an error message explaining what's going on. I'm not filing this ticket with the intention of claiming that I think enabling this toggle should be allowed in my specific case, but if it's prohibited for a good reason then I'd expect any explanation to be displayed. My apologies if there is one somewhere and I just somehow missed it.

What happened instead?

The toggle can't be toggled on and just silently turns off again. Sorry if I'm missing something obvious here, but it seems like a bug that no info is displayed about this.

Operating system

Linux

Browser information

Firefox 129.0.2 (64-bit)

URL for webapp

app.element.io

Application version

Element version: 1.11.76 Crypto version: Rust SDK 0.7.1 (c8c9d15), Vodozemac 0.6.0

Homeserver

matrix.org

Will you send logs?

No

@ell1e ell1e added the T-Defect label Aug 30, 2024
@dosubot dosubot bot added A-Room-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist labels Aug 30, 2024
@sahil9001
Copy link
Contributor

Can I take this one up? I'm actually starting out with Open Source, so wanted to try my hands on this.

@sahil9001
Copy link
Contributor

I have raised a PR, please check

@ell1e
Copy link
Author

ell1e commented Sep 1, 2024

I'm not a developer on Element, but from briefly scanning the code, does it really show the actual error? It looks a little like it just always claims it's a permission issue. At least in the situation where I tested it, I would be surprised if it was a permission issue although I guess I don't really know.

@sahil9001
Copy link
Contributor

@ell1e Well, if you open up the inspect tab, you will notice that the error comes from 403 request. Though I can change this to more generic error like error.message if you want to.

@ell1e
Copy link
Author

ell1e commented Sep 2, 2024

So is your change checking that it's actually a 403 response? My apologies if it is. But I think whether I happen to be seeing a 403 is less relevant to addressing this properly, but rather what errors are reasonably possible.

@alienarena
Copy link

alienarena commented Sep 28, 2024

Same is happening for me. It just toggles back to off without any error.
I tried the same in FluffyChat, at least there I get a message "Not allowed to publish room". But no clue why. Any clue why publishing a room isn't allowed?

@ell1e
Copy link
Author

ell1e commented Sep 28, 2024

@alienarena apparently room publishing on the matrix.org home server is fully disabled. It's not possible to add a room to the directory, it seems like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Error-Message A-Room-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants