Badlop | 1 Sep 2007 10:57
Picon

Re: priority bug

2007/8/31, sergio <ejabberd <at> sergio.spb.ru>:
> ejabberd allows to set priority outsite char (-128..128) range

Bug and patch submitted to:
https://support.process-one.net/browse/EJAB-348

That patch puts priority = 127 if the user tries to put a very high value.
And puts -128 if the user tries to put a very low value.

Do you see this correct, or should be better to put priority=0 if the
user exceeds the allowed range?
Zbyszek Żółkiewski | 1 Sep 2007 12:27
Picon
Gravatar

Re: priority bug

if i may - just my point of view: in my idea any improper data provided by user should be discarded

On 9/1/07, Badlop < badlop <at> gmail.com> wrote:
2007/8/31, sergio < ejabberd <at> sergio.spb.ru>:
> ejabberd allows to set priority outsite char (-128..128) range

Bug and patch submitted to:
https://support.process-one.net/browse/EJAB-348

That patch puts priority = 127 if the user tries to put a very high value.
And puts -128 if the user tries to put a very low value.

Do you see this correct, or should be better to put priority=0 if the
user exceeds the allowed range?
_______________________________________________
ejabberd mailing list
ejabberd <at> jabber.ru
http://lists.jabber.ru/mailman/listinfo/ejabberd



--
pozdrawiam,
Zbyszek Żółkiewski
_______________________________________________
ejabberd mailing list
ejabberd <at> jabber.ru
http://lists.jabber.ru/mailman/listinfo/ejabberd
Badlop | 1 Sep 2007 16:39
Picon

Re: priority bug

2007/8/31, sergio <ejabberd <at> sergio.spb.ru>:
> ejabberd allows to set priority outsite char (-128..128) range

Mickaël Rémond said in the bug tracker:
> This is not a bug but a feature.
>
> I do not see any good reason to enforce this limitation.
> It was intentionally left open ended.
> Some people are using timestamps for example to make sure that their last resource > is the one that has the
highest priority.
>
> I am ready to hear explaination of what harm this feature could cause.

I bring the discussion from the bug tracker to the mailing list again,
so everybody can participate.

I guess the opinion of  the author of the protocol in question may be
helpful. I'll ping Peter Saint-Andre next monday.

Anyway, I see two solutions:

a) Mark the bug as INVALID or WON'T-FIX and reject the patch. If so, I
strongly recommend to create a new document that describes what
inconsistencies are in ejabberd's implementation of protocols. Publish
it, and add to it any other bug-turns-feature [1]. This way other
Jabber developers will not be surprised of some particular behaviors
in ejabberd which are not compliant with standard protocols. This will
also prevent the bug tracker from being filled with many INVALID bugs.

b) Apply the patch, mark the bug as SOLVED, and also implement a new
option to disable this limitation.

[1] Another protocol incompliance that can be considered a feature is
tracked here:
'Lock created room until configured, unless it's non-MUC or instant'
https://support.process-one.net/browse/EJAB-341
sergio | 3 Sep 2007 12:07
Picon

Re: priority bug

Badlop wrote:
> 2007/8/31, sergio <ejabberd <at> sergio.spb.ru>:
>> ejabberd allows to set priority outsite char (-128..128) range
> 
> Bug and patch submitted to:
> https://support.process-one.net/browse/EJAB-348
> 
> That patch puts priority = 127 if the user tries to put a very high value.
> And puts -128 if the user tries to put a very low value.
> 
> Do you see this correct, or should be better to put priority=0 if the
> user exceeds the allowed range?

imho it's better to reject such priority requests with error..

--
sergio.
Gregory Machin | 3 Sep 2007 15:24
Picon

Re: ejabberd newbee no web admin and can't regiter

ejabberd.log

=INFO REPORT==== 2007-08-30 16:25:10 ===
I(<0.179.0>:ejabberd_listener:90): (#Port<0.350>) Accepted connection
{{192,168,1,196},50966} -> {{192,168,1,240},5280}

=INFO REPORT==== 2007-08-30 16:25:10 ===
I(<0.172.0>:ejabberd_http:76): started: {gen_tcp,#Port<0.350>}

=INFO REPORT==== 2007-08-30 16:25:10 ===
I(<0.385.0>:ejabberd_http:175): (#Port<0.350>) http query: 'GET' /

=INFO REPORT==== 2007-08-30 16:26:41 ===
I(<0.385.0>:ejabberd_http:175): (#Port<0.350>) http query: 'GET' /index

=INFO REPORT==== 2007-08-30 16:26:48 ===
I(<0.385.0>:ejabberd_http:175): (#Port<0.350>) http query: 'GET' /index.htlm

=INFO REPORT==== 2007-08-30 16:26:52 ===
I(<0.385.0>:ejabberd_http:175): (#Port<0.350>) http query: 'GET' /index.htm

=INFO REPORT==== 2007-08-30 16:26:55 ===
I(<0.385.0>:ejabberd_http:175): (#Port<0.350>) http query: 'GET' /
sasl.log

=PROGRESS REPORT==== 30-Aug-2007::16:12:59 ===
          supervisor: {local,ejabberd_sup}
             started: [{pid,<0.341.0>},
                       {name,ejabberd_mod_pubsub_localhost},
                       {mfa,{mod_pubsub,
                                start_link,
                                ["localhost",
                                 [{access_createnode,pubsub_createnode}]]}},
                       {restart_type,temporary},
                       {shutdown,1000},
                       {child_type,worker}]

=PROGRESS REPORT==== 30-Aug-2007::16:12:59 ===
         application: ejabberd
          started_at: ejabberd <at> ns1
[root <at> ns1 ~]#

many thanks for your time

On 8/31/07, Magnus Henoch <mange <at> freemail.hu> wrote:
> "Gregory Machin" <gregory.machin <at> gmail.com> writes:
>
> > Aug 31 14:00:28 ns1 epmd: epmd: epmd running - daemon = 1
> > Aug 31 14:00:45 ns1 epmd: epmd: epmd running - daemon = 1
> > Aug 31 14:00:48 ns1 epmd: epmd: node name already occupied ejabberd
>
> That's the syslog, right?  ejabberd has two own log files, usually
> called ejabberd.log and sasl.log.  What do they say?
>
> --
> Magnus
> JID: legoscia <at> jabber.cd.chalmers.se
>
> _______________________________________________
> ejabberd mailing list
> ejabberd <at> jabber.ru
> http://lists.jabber.ru/mailman/listinfo/ejabberd
>

--

-- 
Gregory Machin
gregory.machin <at> gmail.com
www.linuxpro.co.za
Jeremy Chow | 4 Sep 2007 11:58
Picon
Gravatar

how does ejabberd_c2s supervisor start?

Hi all,
   ejabberd_c2s_sup  is defined in ejabberd_sup, deletegated by ejabberd_tmp_sup with a restart strategie simple_one_for_one. As the otp document indicates, the child process , ejabberd_c2s_sup in this case, will be added and be started dynamically by supervisor:start_child/2. I use start_child as keyword for searching, and found a function ejabberd_c2s:start/2  would call that one. My question is who call the ejabberd_c2s:start/2 in the standalone version of ejabberd without http(another  way to start this service explicitly is in ejabberd_http_poll module).


Thx,
Jeremy

_______________________________________________
ejabberd mailing list
ejabberd <at> jabber.ru
http://lists.jabber.ru/mailman/listinfo/ejabberd
Pedro Melo | 4 Sep 2007 17:14
Favicon
Gravatar

Next release time frame

Hi,

I need to update a server from OpenFire to ejabberd.

Via fisheye, I see that a 1.1.4 release is being prepared.

Any ideas when to expect that? I can wait a couple of days before  
upgrading...

Thanks in advance,
--

-- 
Pedro Melo
Blog: http://www.simplicidade.org/notes/
XMPP ID: melo <at> simplicidade.org
Use XMPP!
Peter Saint-Andre | 4 Sep 2007 20:56
Favicon

Re: priority bug

Badlop wrote:
> 2007/8/31, sergio <ejabberd <at> sergio.spb.ru>:
>> ejabberd allows to set priority outsite char (-128..128) range
> 
> Mickaël Rémond said in the bug tracker:
>> This is not a bug but a feature.
>> 
>> I do not see any good reason to enforce this limitation. It was
>> intentionally left open ended. Some people are using timestamps for
>> example to make sure that their last resource > is the one that has
>> the highest priority.
>> 
>> I am ready to hear explaination of what harm this feature could
>> cause.
> 
> I bring the discussion from the bug tracker to the mailing list
> again, so everybody can participate.
> 
> I guess the opinion of  the author of the protocol in question may be
>  helpful. I'll ping Peter Saint-Andre next monday.

RFC 3921 and rfc3921bis state:

   The OPTIONAL <priority/> element contains non-human-readable XML
   character data that specifies the priority level of the resource.
   The value MUST be an integer between -128 and +127.

If people want some special functionality, we can define a new child
element in a different namespace. After all, the "X" in XMPP stands for
"Extensible". :)

> Anyway, I see two solutions:
> 
> a) Mark the bug as INVALID or WON'T-FIX and reject the patch. If so,
> I strongly recommend to create a new document that describes what 
> inconsistencies are in ejabberd's implementation of protocols.
> Publish it, and add to it any other bug-turns-feature [1]. This way
> other Jabber developers will not be surprised of some particular
> behaviors in ejabberd which are not compliant with standard
> protocols. This will also prevent the bug tracker from being filled
> with many INVALID bugs.
> 
> b) Apply the patch, mark the bug as SOLVED, and also implement a new 
> option to disable this limitation.

I would be in favor of a new protocol extension that enables more
flexible presence priorities / stanza routing. Overloading priority
isn't a good solution.

Peter

--

-- 
Peter Saint-Andre
https://stpeter.im/

Attachment (smime.p7s): application/x-pkcs7-signature, 7338 bytes
_______________________________________________
ejabberd mailing list
ejabberd <at> jabber.ru
http://lists.jabber.ru/mailman/listinfo/ejabberd
Sander Devrieze | 4 Sep 2007 21:34
Picon

Re: priority bug

2007/9/4, Peter Saint-Andre <stpeter <at> stpeter.im>:
> I would be in favor of a new protocol extension that enables more
> flexible presence priorities / stanza routing. Overloading priority
> isn't a good solution.

Maybe an idea for a PEP XEP called "User Priority" ;-)

--

-- 
Mvg, Sander Devrieze.
Christophe Romain | 5 Sep 2007 10:55
Favicon

Re: Next release time frame

Hi

ejabberd-1.1.4 is released and installers are available
http://www.process-one.net/en/ejabberd/downloads

regards.

Gmane