You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openmeetings.apache.org by "seba.wagner@gmail.com" <se...@gmail.com> on 2013/02/11 20:52:43 UTC

Re: svn commit: r1444703 - /openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java

Hi Maxim,

What does two way sound mean?
Is it a Sip specific thing or do you open several streams?
Just curious...

THX!
Seb
Am 11.02.2013 23:02 schrieb <so...@apache.org>:

> Author: solomax
> Date: Mon Feb 11 10:01:43 2013
> New Revision: 1444703
>
> URL: http://svn.apache.org/r1444703
> Log:
> SIP: nat column is added to sipusers table to enable 2-way sound in SIP.
>
> Modified:
>
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
>
> Modified:
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> URL:
> http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java?rev=1444703&r1=1444702&r2=1444703&view=diff
>
> ==============================================================================
> ---
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> (original)
> +++
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> Mon Feb 11 10:01:43 2013
> @@ -98,6 +98,10 @@ public class AsteriskSipUser implements
>         @Element(data=true, required = false)
>         private String md5secret;
>
> +       @Column(name = "nat", nullable=false)
> +       @Element(data=true, required = false)
> +       private String nat = "yes";
> +
>         public long getId() {
>                 return id;
>         }
> @@ -217,4 +221,12 @@ public class AsteriskSipUser implements
>         public void setMd5secret(String md5secret) {
>                 this.md5secret = md5secret;
>         }
> +
> +       public String getNat() {
> +               return nat;
> +       }
> +
> +       public void setNat(String nat) {
> +               this.nat = nat;
> +       }
>  }
> \ No newline at end of file
>
>
>

Re: svn commit: r1444703 - /openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java

Posted by Maxim Solodovnik <so...@gmail.com>.
It was really weird

without "nat=yes" user setting I can hear everybody in the room, but nobody
can hear me :(

The only issue with the OM now is:
if there is lot of users lots of users in the room (5-7) some of them can't
hear some others until "reload" of streams on the client.
Timur told me that according to the log of red5sip there is weird queue of
method calls: newStream gets called twice. Additionally if user has stream
with id 24 and select "reload" the following methods are gets called:
closeStream(25) // NOTE 25 not 24
then
newStream(25)

Maybe you can say what is wrong?



On Tue, Feb 12, 2013 at 2:52 AM, seba.wagner@gmail.com <
seba.wagner@gmail.com> wrote:

> Hi Maxim,
>
> What does two way sound mean?
> Is it a Sip specific thing or do you open several streams?
> Just curious...
>
> THX!
> Seb
> Am 11.02.2013 23:02 schrieb <so...@apache.org>:
>
> > Author: solomax
> > Date: Mon Feb 11 10:01:43 2013
> > New Revision: 1444703
> >
> > URL: http://svn.apache.org/r1444703
> > Log:
> > SIP: nat column is added to sipusers table to enable 2-way sound in SIP.
> >
> > Modified:
> >
> >
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> >
> > Modified:
> >
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> > URL:
> >
> http://svn.apache.org/viewvc/openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java?rev=1444703&r1=1444702&r2=1444703&view=diff
> >
> >
> ==============================================================================
> > ---
> >
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> > (original)
> > +++
> >
> openmeetings/trunk/singlewebapp/src/org/apache/openmeetings/persistence/beans/sip/asterisk/AsteriskSipUser.java
> > Mon Feb 11 10:01:43 2013
> > @@ -98,6 +98,10 @@ public class AsteriskSipUser implements
> >         @Element(data=true, required = false)
> >         private String md5secret;
> >
> > +       @Column(name = "nat", nullable=false)
> > +       @Element(data=true, required = false)
> > +       private String nat = "yes";
> > +
> >         public long getId() {
> >                 return id;
> >         }
> > @@ -217,4 +221,12 @@ public class AsteriskSipUser implements
> >         public void setMd5secret(String md5secret) {
> >                 this.md5secret = md5secret;
> >         }
> > +
> > +       public String getNat() {
> > +               return nat;
> > +       }
> > +
> > +       public void setNat(String nat) {
> > +               this.nat = nat;
> > +       }
> >  }
> > \ No newline at end of file
> >
> >
> >
>



-- 
WBR
Maxim aka solomax