search

nioc / xmpp-web

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

Add staging environments #89

Closed nioc closed 1 year ago

nioc commented 1 year ago

Staging server setups are provided in order to be able to reproduce bugs more easily.

nioc commented 1 year ago

Hello dear XMPP experts :bowtie: (@Ppjet6 @poVoq @singpolyma), I would like you review this Prosody setup.

Feel free to complete (comment here or commit in this branch).

Ppjet6 commented 1 year ago

As this is for testing purposes only (right?), you may want to have a look at: https://modules.prosody.im/mod_auth_any.html

Other than this, a few quick comments:

disco_items is filled up automatically by prosody for each virtualhost with components configured on subdomains of that virtualhost, so you don't need to declare it here. But it's ok if you want it explicit.

proxy65 and http_upload are loaded as global modules and will be loaded automatically on your localhost virtualhost. The additional component for proxy65 is redundant (and the one for http_upload was commented out also). You can move the proxy65 settings to the virtualhost, or globally like you did for http_upload. And you can also remove them from disco items.

You can probably set s2s_require_encryption = false, or disable s2s altogether.

Hopefully the next Debian release should be there soon(tm), and you'll be able to have it in stable. It's only available in bullseye-backports currently.

nioc commented 1 year ago

Thanks a lot for your time.

Yes, it is for test and debug only, not for production 😁

I answer to your points as comment to check if I understand correctly.

nioc commented 1 year ago

If this is ok for you, I will add a reproducing procedure and merge the pull request tonight. Thanks for your feedback 👍🏻