ejabberd - Comments for "Unable to include both xmpp.hrl and jlib.hrl"

I believe the problems I'm

nosnilmot — Wed, 26 Jul 2017 11:25:33 +0000

I believe the problems I'm seeing with adding an IQ handler are specific to when my module is loaded only for a specific virtual host. If I load it globally, mod_ping is not affected. I suspect this may be an ejabberd bug.

Thank you for your assistance

nosnilmot — Tue, 25 Jul 2017 14:39:25 +0000

Thank you for your assistance so far, I think I am nearly where I want to be, but it appears that my custom IQ handler is causing problems for others.

For example, with my module loaded and having answered one query, mod_ping is no longer processing client pings, and clients are being returned 503 / "No module is handling this query" errors.

See

zinid — Sun, 23 Jul 2017 10:03:22 +0000

See https://github.com/processone/xmpp/issues/9

Do I need to modify XMPP

nosnilmot — Thu, 20 Jul 2017 10:20:00 +0000

Do I need to modify XMPP library to add support for a new namespace for IQ queries?

I did have this part working before, when I was using jlib.hrl ...

2017-07-20 11:10:38.475 [debug] <0.1612.0>@ejabberd_receiver:process_data:284 Received XML on stream = <<"<iq to='localhost' type='get' id='123'>\n<query xmlns='urn:xmpp:helloworld'/>\n</iq>">>
...
2017-07-20 11:10:38.480 [debug] <0.1613.0>@ejabberd_socket:send:216 (tcp|<0.1612.0>) Send XML on stream = <<"<iq xml:lang='en' to='stu@localhost/resource' from='localhost' type='error' id='123'><query xmlns='urn:xmpp:helloworld'/><error code='400' type='modify'><bad-request xmlns='urn:ietf:params:xml:ns:xmpp-stanzas'/><text xml:lang='en' xmlns='urn:ietf:params:xml:ns:xmpp-stanzas'>Unknown tag &lt;query/&gt; qualified by namespace &apos;urn:xmpp:helloworld&apos;</text></error></iq>">>

You cannot use both xmpp.hrl

zinid — Thu, 20 Jul 2017 07:16:13 +0000

You cannot use both xmpp.hrl and jlib.hrl.
Also, jlib.hrl is deprecated and actually is forbidden to use. Use xmpp.hrl only. Adapt your code using new xmpp library, you can find the library's description here: https://github.com/processone/xmpp/blob/master/README.md

using xmpp:make_error(IQ,

nosnilmot — Wed, 19 Jul 2017 12:04:12 +0000

using xmpp:make_error(IQ, xmpp:err_not_authorized()) allows me to avoid that error and get one step towards not including jlib.hrl (I presume that is the goal here?), but now I have these when I don't include jlib.hrl:

/Users/stu/.ejabberd-modules/sources/mod_hello_world/src/mod_hello_world.erl:47: field sub_el undefined in record iq
/Users/stu/.ejabberd-modules/sources/mod_hello_world/src/mod_hello_world.erl:47: field xmlns undefined in record iq

Don't use ?ERR_... Search in

badlop — Wed, 19 Jul 2017 09:42:10 +0000

Don't use ?ERR_...

Search in ejabberd source code examples of how to use xmpp:err_not_authorized(ErrText, Lang),