xmpp-cloud-auth icon indicating copy to clipboard operation
xmpp-cloud-auth copied to clipboard

Documentation not updated for .deb version?

Open chriscroome opened this issue 6 years ago • 7 comments

I'm looking at upgrading some Nextcloud server which are running version v1.1.0 (code downloaded using a git clone) to v2.0.2 (installed using the .deb package) and it appears to me that the documentation hasn't been fully updated to match the use of the .deb version?

For example, which service should be started and set to run after the package has been installed?

The xcauth.py file either no longer exists or had been renamed?

chriscroome avatar Sep 27 '18 10:09 chriscroome

I might be able to answer some of my own questions...

To start the service:

service xcauth start

chriscroome avatar Sep 27 '18 10:09 chriscroome

When starting the service there is this issue:

  Process: 22599 ExecStartPre=/bin/chgrp sasl /var/run/saslauthd/ (code=exited, status=1/FAILURE)

This is on a server that won't use saslauthd so it hasn't been installed...

chriscroome avatar Sep 27 '18 10:09 chriscroome

If this line in /etc/systemd/system/xcauth.service is commented out:

#ExecStartPre=-+/bin/chgrp sasl /var/run/saslauthd/

And then you run:

systemctl daemon-reload
service xcauth status

Then you don't get the issue above.

chriscroome avatar Sep 27 '18 17:09 chriscroome

Thanks for reporting this issue. @MarcelWaldvogel will look into this as soon as possible.

sualko avatar Sep 28 '18 07:09 sualko

The steps I took to get the .deb working were:

  1. Symlink the files in /usr/lib/prosody/modules/xcauth/ from /usr/lib/prosody/modules/ (omitting .lua-xcauth-version from mod_auth_external).
  2. Change the socket port in /etc/prosody/prosody.cfg.lua to external_auth_command = "@localhost:23663" (previously it was port 23664 as this is the documented port).
  3. Comment out the line ExecStartPre=-+/bin/chgrp sasl /var/run/saslauthd/ in /etc/systemd/system/xcauth.service since I'm not using SASL authentication on Nextcloud servers.

Apart from the documentation needed updating I don't understand why these files:

dpkg -L xcauth | grep lua
/usr/lib/prosody/modules/xcauth/mod_auth_external.lua-xcauth-version
/usr/lib/prosody/modules/xcauth/pseudolpty.lib.lua

Are not written to the /usr/lib/prosody/modules/ directory since there are no files that they would clobber provided by the Prosody .deb from their repo (perhaps other packages do have files of the same names?):

dpkg -L prosody | grep modules
/usr/lib/prosody/modules
/usr/lib/prosody/modules/adhoc
/usr/lib/prosody/modules/adhoc/adhoc.lib.lua
/usr/lib/prosody/modules/adhoc/mod_adhoc.lua
/usr/lib/prosody/modules/mod_admin_adhoc.lua
/usr/lib/prosody/modules/mod_admin_telnet.lua
/usr/lib/prosody/modules/mod_announce.lua
/usr/lib/prosody/modules/mod_auth_anonymous.lua
/usr/lib/prosody/modules/mod_auth_cyrus.lua
/usr/lib/prosody/modules/mod_auth_internal_hashed.lua
/usr/lib/prosody/modules/mod_auth_internal_plain.lua
/usr/lib/prosody/modules/mod_blocklist.lua
/usr/lib/prosody/modules/mod_bosh.lua
/usr/lib/prosody/modules/mod_c2s.lua
/usr/lib/prosody/modules/mod_carbons.lua
/usr/lib/prosody/modules/mod_component.lua
/usr/lib/prosody/modules/mod_compression.lua
/usr/lib/prosody/modules/mod_debug_sql.lua
/usr/lib/prosody/modules/mod_dialback.lua
/usr/lib/prosody/modules/mod_disco.lua
/usr/lib/prosody/modules/mod_groups.lua
/usr/lib/prosody/modules/mod_http.lua
/usr/lib/prosody/modules/mod_http_errors.lua
/usr/lib/prosody/modules/mod_http_files.lua
/usr/lib/prosody/modules/mod_iq.lua
/usr/lib/prosody/modules/mod_lastactivity.lua
/usr/lib/prosody/modules/mod_legacyauth.lua
/usr/lib/prosody/modules/mod_limits.lua
/usr/lib/prosody/modules/mod_mam
/usr/lib/prosody/modules/mod_mam/fallback_archive.lib.lua
/usr/lib/prosody/modules/mod_mam/mamprefs.lib.lua
/usr/lib/prosody/modules/mod_mam/mamprefsxml.lib.lua
/usr/lib/prosody/modules/mod_mam/mod_mam.lua
/usr/lib/prosody/modules/mod_message.lua
/usr/lib/prosody/modules/mod_motd.lua
/usr/lib/prosody/modules/mod_net_multiplex.lua
/usr/lib/prosody/modules/mod_offline.lua
/usr/lib/prosody/modules/mod_pep.lua
/usr/lib/prosody/modules/mod_ping.lua
/usr/lib/prosody/modules/mod_posix.lua
/usr/lib/prosody/modules/mod_presence.lua
/usr/lib/prosody/modules/mod_privacy.lua
/usr/lib/prosody/modules/mod_private.lua
/usr/lib/prosody/modules/mod_proxy65.lua
/usr/lib/prosody/modules/mod_pubsub
/usr/lib/prosody/modules/mod_pubsub/mod_pubsub.lua
/usr/lib/prosody/modules/mod_pubsub/pubsub.lib.lua
/usr/lib/prosody/modules/mod_register.lua
/usr/lib/prosody/modules/mod_roster.lua
/usr/lib/prosody/modules/mod_s2s
/usr/lib/prosody/modules/mod_s2s/mod_s2s.lua
/usr/lib/prosody/modules/mod_s2s/s2sout.lib.lua
/usr/lib/prosody/modules/mod_s2s_auth_certs.lua
/usr/lib/prosody/modules/mod_saslauth.lua
/usr/lib/prosody/modules/mod_server_contact_info.lua
/usr/lib/prosody/modules/mod_stanza_debug.lua
/usr/lib/prosody/modules/mod_storage_internal.lua
/usr/lib/prosody/modules/mod_storage_none.lua
/usr/lib/prosody/modules/mod_storage_sql.lua
/usr/lib/prosody/modules/mod_storage_sql1.lua
/usr/lib/prosody/modules/mod_storage_xep0227.lua
/usr/lib/prosody/modules/mod_time.lua
/usr/lib/prosody/modules/mod_tls.lua
/usr/lib/prosody/modules/mod_unknown.lua
/usr/lib/prosody/modules/mod_uptime.lua
/usr/lib/prosody/modules/mod_vcard.lua
/usr/lib/prosody/modules/mod_version.lua
/usr/lib/prosody/modules/mod_watchregistrations.lua
/usr/lib/prosody/modules/mod_websocket.lua
/usr/lib/prosody/modules/mod_welcome.lua
/usr/lib/prosody/modules/mod_windows.lua
/usr/lib/prosody/modules/muc
/usr/lib/prosody/modules/muc/mod_muc.lua
/usr/lib/prosody/modules/muc/muc.lib.lua

chriscroome avatar Sep 28 '18 10:09 chriscroome

I will look into this in the next few days

MarcelWaldvogel avatar Oct 05 '18 08:10 MarcelWaldvogel

Oops, that fell under the radar somehow.

  • ExecStartPre=: The - should imply that systemd should not be bothered by errors. Does it actually not work or is just the error message bothering you?
  • mod_auth_external.lua: I do not currently have a Prosody installation; but I thought that it came in some module package. If this is not true, I'll happily change this.
  • Port number documentation fixed in 4e84b602465

MarcelWaldvogel avatar Sep 19 '19 14:09 MarcelWaldvogel