ejabberd - Comments for "how to get users "os" in module mod_muc_room"

How can I do it? thnx.

davai — Tue, 03 Nov 2015 15:13:52 +0000

How can I do it? thnx.

Hi; Thank u I edited

davai — Tue, 03 Nov 2015 14:37:30 +0000

Hi;

Thank u

I edited mod_muc_room like what u say

and add get_os function

%% get OS
get_os(Packet) ->
	Q = xml:get_subtag(Packet, "query"),
	O = xml:get_subtag(Q, "os"),
	{ok, xml:get_tag_cdata(O)}.

trying :

    Os = case catch get_os(Packet) of
       {ok, Result} -> Result;
       _ -> not_iq_version
    end,
    ?INFO_MSG("OS is : ~p", [OS]),

also trying :

    Os = case catch get_os(Packet) of
       {ok, Result} -> Result;
       _ -> not_iq_version
    end,
    ?WARNING_MSG("OS is : ~p", [OS]),

so I run Ejabberd : ejabberdctl live. and when I'm joining to room no thing show to me in terminal.

I'm using Ejabberd 2.1.x

I'm using ejabberd 15.09. The

badlop — Mon, 19 Oct 2015 10:18:05 +0000

I'm using ejabberd 15.09. The code is similar for older versions, you can reuse it.

This patch:

--- a/src/mod_muc_room.erl
+++ b/src/mod_muc_room.erl
@@ -618,6 +618,11 @@ normal_state({route, From, ToNick,
             StateData) ->
     Lang = xml:get_attr_s(<<"xml:lang">>, Attrs),
     StanzaId = xml:get_attr_s(<<"id">>, Attrs),
+    Os = case catch get_os(Packet) of
+       {ok, Result} -> Result;
+       _ -> not_iq_version
+    end,
+    ?INFO_MSG("Os: ~p", [Os]),
     case {(StateData#state.config)#config.allow_query_users,
          is_user_online_iq(StanzaId, From, StateData)}
        of
@@ -4539,3 +4544,7 @@ has_body_or_subject(Packet) ->
        (_) -> true
     end, Packet#xmlel.children).
 
+get_os(Packet) ->
+    Q = xml:get_subtag(Packet, <<"query">>),
+    O = xml:get_subtag(Q, <<"os">>),
+    {ok, xml:get_tag_cdata(O)}.

And I get something like this in the log:

12:15:58.841 [info] ({socket_state,gen_tcp,#Port<0.8631>,<0.507.0>}) Accepted legacy authentication for user1@localhost/tka1 by ejabberd_auth_internal from 127.0.0.1
12:16:06.458 [info] Os: not_iq_version
12:16:06.461 [info] Os: not_iq_version
12:16:06.466 [info] Os: not_iq_version
12:16:06.466 [info] Os: not_iq_version
12:16:06.471 [info] Os: <<"Debian GNU/Linux 8.1 (jessie) 8.1 jessie 4.1.0-2-amd64">>
12:16:06.472 [info] Os: not_iq_version
12:16:06.475 [info] Os: not_iq_version
12:16:06.477 [info] Os: not_iq_version

Hi; Sorry for late. Xmpp

davai — Fri, 16 Oct 2015 01:15:43 +0000

Hi;

Sorry for late.

Xmpp packet like this :

<query xmlns='http://jabber.org/protocol/disco#info'
         node='http://psi-im.org#q07IKJEyjvHSyhy//CH0CxmKi8w='>
    <identity xml:lang='en' category='client' name='Psi 0.11' type='pc'/>
    <identity xml:lang='el' category='client' name='Ψ 0.11' type='pc'/>
    <feature var='http://jabber.org/protocol/caps'/>
    <feature var='http://jabber.org/protocol/disco#info'/>
    <feature var='http://jabber.org/protocol/disco#items'/>
    <feature var='http://jabber.org/protocol/muc'/>
    <x xmlns='jabber:x:data' type='result'>
      <field var='FORM_TYPE' type='hidden'>
        <value>urn:xmpp:dataforms:softwareinfo</value>
      </field>
      <field var='ip_version'>
        <value>ipv4</value>
        <value>ipv6</value>
      </field>
      <field var='os'>
        <value>Windows 8.1</value>
      </field>
      <field var='os_version'>
        <value>10.5.1</value>
      </field>
      <field var='software'>
        <value>Psi</value>
      </field>
      <field var='software_version'>
        <value>0.11</value>
      </field>
    </x>
</query>

What I'm looking for is get os like : Windows 8.1

<field var='os'>
   <value>Windows 8.1</value><

An XMPP Packet is the basis

mremond — Mon, 05 Oct 2015 09:20:24 +0000

An XMPP Packet is the basis of XMPP protocol. You should read more on xmpp.org.

mremond wrote: It would be

davai — Wed, 30 Sep 2015 22:01:04 +0000

mremond wrote:

It would be easier to help if you could show the XMPP packet you are trying to parse. Can you please do that ?

Hi;

The idea is like any bot (iSida, gluxi, sulci ... etc) if u type to him this : version mremond

it'll send u your version information like -->

name : Psi+
ver : 1.1
os : windows 8.1

Example --> I need this os : windows 8.1 to print it when mremond joining the room, I need to definition the OS

I don't know what do u mean the XMPP packet in this case can u explain me with some example please!

thank u

It would be easier to help if

mremond — Wed, 30 Sep 2015 16:18:50 +0000

It would be easier to help if you could show the XMPP packet you are trying to parse. Can you please do that ?

mremond wrote: More

davai — Tue, 29 Sep 2015 20:08:08 +0000

mremond wrote:

More specifically: Is your code crashing ? Is your OS tag not found ? What is your original XMPP packet looking like exactly ?

Hi;

yes, OS tag not found. because nothing shown on terminal when I join.
I think this : OS = xml:get_subtag_cdata(Packet, "os"), not the correct way to definition the OS

More specifically: Is your

mremond — Tue, 29 Sep 2015 07:21:00 +0000

More specifically: Is your code crashing ? Is your OS tag not found ? What is your original XMPP packet looking like exactly ?

mremond wrote: What does not

davai — Sun, 27 Sep 2015 23:46:29 +0000

mremond wrote:

What does not work ? What are you trying to achieve ?

I'm trying to define OS in mod_muc_room to get user's OS before joining room and print it.

What does not work ? What are

mremond — Thu, 24 Sep 2015 08:57:11 +0000

What does not work ? What are you trying to achieve ?

ejabberd - Comments for "how to get users &quot;os&quot; in module mod_muc_room"

How can I do it? thnx.

Hi; Thank u I edited

I'm using ejabberd 15.09. The

Hi; Sorry for late. Xmpp

An XMPP Packet is the basis

mremond wrote: It would be

It would be easier to help if

mremond wrote: More

More specifically: Is your

mremond wrote: What does not

What does not work ? What are

ejabberd - Comments for "how to get users "os" in module mod_muc_room"