nioc / xmpp-web

Lightweight web chat client for XMPP server
GNU Affero General Public License v3.0
144 stars 20 forks source link

Configurable homepage route #23

Closed southerntofu closed 3 years ago

southerntofu commented 3 years ago

Currently, homepage route is for non-anonymous login. I would like to use /guest/ route as default for index.html.

nioc commented 3 years ago

This is a very specific usage 😆.

But may be I can set the routes according to a local.js variable to allow or not routes/components for:

southerntofu commented 3 years ago

This is a very specific usage laughing.

For the moment it is indeed. But a lot of people just want a chatroom for guests, for example for discussion around a free-software projects. Moderators already have a desktop/TUI client, but need a quick way for guest users to come and ask a question from their web browser, like many do on gitter/discord currently. Do you think it's a valid usecase?

In particular when there is no existing trust relationship between the client and the server, i find it more reasonable to disable "real" logins entirely, because we've tried for two decades now to educate users they should not give away their passwords to random services on the web ;-)

But may be I can set the routes according to a local.js variable to allow or not routes/components

That would be perfect, thanks!

nioc commented 3 years ago

Agreed for the project/support use case and understanding the security argument behind your request. May be, it will require to add some limitation/quota server side but it is another subject...

southerntofu commented 3 years ago

Hello and thanks for the quick release of v0.8.0! Sorry to "reopen" this issue but it appears the default route has weird behavior:

No idea why but maybe http://localhost/#/rooms/welcome@conference.localhost (URI fragments for internal routes) like you mentioned in #21 could help prevent that kind of inconsistencies.

nioc commented 3 years ago

May be a cache issue. When I open your url, I have the nick form for guest access.

southerntofu commented 3 years ago

May be a cache issue.

Thanks, that was it! Weird it happened despite explicitly asking for a no-cache refresh (ctrl + f5). Thanks again for the quick release, take care