Re: [asterisk-dev] [Code Review] 2994: ari:Add application/json parameter support

2013-11-27 Thread David Lee
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/2994/ --- (Updated Nov. 27, 2013, 9:39 a.m.) Status -- This change has been

Re: [asterisk-dev] [Code Review] 2994: ari:Add application/json parameter support

2013-11-26 Thread Matt Jordan
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/2994/#review10286 --- Ship it! Ship It! - Matt Jordan On Nov. 21, 2013, 8:47

Re: [asterisk-dev] [Code Review] 2994: ari:Add application/json parameter support

2013-11-21 Thread David Lee
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/2994/ --- (Updated Nov. 21, 2013, 2:47 p.m.) Review request for Asterisk Developers

Re: [asterisk-dev] [Code Review] 2994: ari:Add application/json parameter support

2013-11-21 Thread David Lee
On Nov. 20, 2013, 6:07 p.m., Matt Jordan wrote: /branches/12/main/http.c, line 631 https://reviewboard.asterisk.org/r/2994/diff/2/?file=48312#file48312line631 Particularly since this is coming from an external source, we should probably use sscanf. You pretty much do