Re: [asterisk-dev] Wrong entity field in NOTIFY??

2014-05-09 Thread Olle E. Johansson
On 08 May 2014, at 18:33, Eugen Dedu eugen.d...@pu-pm.univ-fcomte.fr wrote: Hi asterisk devs, Since several years we (opal, ekiga VoIP community) noticed that there is a problem in the NOTIFY messages created by asterisk (entity attribute is set to the subscriber instead of the

Re: [asterisk-dev] Wrong entity field in NOTIFY??

2014-05-09 Thread Eugen Dedu
On 09/05/14 09:31, Olle E. Johansson wrote: On 08 May 2014, at 18:33, Eugen Dedueugen.d...@pu-pm.univ-fcomte.fr wrote: Hi asterisk devs, Since several years we (opal, ekiga VoIP community) noticed that there is a problem in the NOTIFY messages created by asterisk (entity attribute is set to

Re: [asterisk-dev] [Code Review] 3505: app_chanspy: Fix a bug where barge mode only works on the first connection when multiple sessions are spied on for a channel

2014-05-09 Thread Jonathan Rose
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated May 9, 2014, 11:10 a.m.) Status -- This change has been

[asterisk-dev] [Code Review] 3525: Testsuite: Fix ARI attended transfer test

2014-05-09 Thread opticron
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3525/ --- Review request for Asterisk Developers and Joshua Colp. Bugs:

Re: [asterisk-dev] [Code Review] 3494: ARI: Add the ability to raise an arbitrary User Event from the Applications resource

2014-05-09 Thread Scott Griepentrog
On May 1, 2014, 2:26 p.m., Matt Jordan wrote: /branches/12/include/asterisk/stasis_app.h, lines 242-247 https://reviewboard.asterisk.org/r/3494/diff/1/?file=58104#file58104line242 Why AGER? App_generate? I'd use APP_GEN, personally. It's a bit easier to read. I was

Re: [asterisk-dev] [Code Review] 3494: ARI: Add the ability to raise an arbitrary User Event from the Applications resource

2014-05-09 Thread Scott Griepentrog
On May 2, 2014, 8:37 a.m., Matt Jordan wrote: /branches/12/main/stasis_user.c, lines 64-74 https://reviewboard.asterisk.org/r/3494/diff/1/?file=58110#file58110line64 Wrapping the snapshots is fine, but I think all you'll need is the type stored as an enum. That will allow

Re: [asterisk-dev] [Code Review] 3494: ARI: Add the ability to raise an arbitrary User Event from the Applications resource

2014-05-09 Thread Scott Griepentrog
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3494/ --- (Updated May 9, 2014, 5:29 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3494: ARI: Add the ability to raise an arbitrary User Event from the Applications resource

2014-05-09 Thread Scott Griepentrog
On May 1, 2014, 4:59 p.m., Samuel Galarneau wrote: /branches/12/rest-api/api-docs/applications.json, line 51 https://reviewboard.asterisk.org/r/3494/diff/1/?file=58118#file58118line51 What was the thought process behind adding this API endpoint to the applications resource? Would