<div dir="ltr">Style nit:&nbsp; try to respect the local style of the file (ideally: project) you&#39;re hacking in.&nbsp; (one of the <a href="http://danga.com/commit.bml">danga.com/commit.bml</a> rules)<br><br>Notably, this is inconsistent with the rest of the file:<br>
<br>
+ &nbsp; &nbsp;unless( $ss-&gt;xmlns eq $self-&gt;namespace ) {<br><br>Should be:<br><br>
+ &nbsp; &nbsp;unless ($ss-&gt;xmlns eq $self-&gt;namespace) {<br><br><br><div class="gmail_quote">On Sun, Jul 27, 2008 at 3:11 PM,  <span dir="ltr">&lt;<a href="mailto:commits@code.sixapart.com">commits@code.sixapart.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">&nbsp;r6512@coke: &nbsp;josboum | 2008-07-27 01:07:51 +0200<br>
&nbsp;* log stream errors clearly and don&#39;t just silently disconnect<br>
&nbsp;* this addresses a FIXME<br>
<br>
<br>
_U &nbsp;trunk/<br>
U &nbsp; trunk/DJabberd/lib/DJabberd/Connection/ServerIn.pm<br>
<br>
<br>
<br>
Property changes on: trunk<br>
___________________________________________________________________<br>
Name: svk:merge<br>
 &nbsp; - 02e31fea-a713-0410-9bb5-81a53d0f95f7:/djabberd-trunk:4879<br>
1b5ff1af-660f-0410-86e2-35bade70ffec:/trunk/djabberd:61<br>
391f84f8-cfa2-4f6b-a47d-f52c5dc88d86:/local/sixapart/djabberd-trunk:6511<br>
54bb65a0-b99f-4f2b-b22a-e47d41dc43c7:/groupchat:328<br>
54bb65a0-b99f-4f2b-b22a-e47d41dc43c7:/local:667<br>
 &nbsp; + 02e31fea-a713-0410-9bb5-81a53d0f95f7:/djabberd-trunk:4879<br>
1b5ff1af-660f-0410-86e2-35bade70ffec:/trunk/djabberd:61<br>
391f84f8-cfa2-4f6b-a47d-f52c5dc88d86:/local/sixapart/djabberd-trunk:6512<br>
54bb65a0-b99f-4f2b-b22a-e47d41dc43c7:/groupchat:328<br>
54bb65a0-b99f-4f2b-b22a-e47d41dc43c7:/local:667<br>
<br>
Modified: trunk/DJabberd/lib/DJabberd/Connection/ServerIn.pm<br>
===================================================================<br>
--- trunk/DJabberd/lib/DJabberd/Connection/ServerIn.pm &nbsp;2008-07-27 22:10:59 UTC (rev 780)<br>
+++ trunk/DJabberd/lib/DJabberd/Connection/ServerIn.pm &nbsp;2008-07-27 22:11:09 UTC (rev 781)<br>
@@ -22,8 +22,16 @@<br>
&nbsp;sub on_stream_start {<br>
 &nbsp; &nbsp; my ($self, $ss) = @_;<br>
 &nbsp; &nbsp; $self-&gt;{in_stream} = 1;<br>
- &nbsp; &nbsp;return $self-&gt;close unless $ss-&gt;xmlns eq $self-&gt;namespace; # FIXME: should be stream error<br>
<br>
+ &nbsp; &nbsp;### namespace mismatch is a streamerror<br>
+ &nbsp; &nbsp;unless( $ss-&gt;xmlns eq $self-&gt;namespace ) {<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;$self-&gt;stream_error(<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;sprintf &quot;namespace mismatch: client-&gt;%s server-&gt;%s&quot;,<br>
+ &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;$ss-&gt;xmlns, $self-&gt;namespace<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;);<br>
+ &nbsp; &nbsp; &nbsp; &nbsp;$self-&gt;close;<br>
+ &nbsp; &nbsp;}<br>
+<br>
 &nbsp; &nbsp; if ($ss-&gt;announced_dialback) {<br>
 &nbsp; &nbsp; &nbsp; &nbsp; $self-&gt;{announced_dialback} = 1;<br>
 &nbsp; &nbsp; &nbsp; &nbsp; $self-&gt;start_stream_back($ss,<br>
<br>
<br>
</blockquote></div><br></div>