----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4519/#review14792 -----------------------------------------------------------
./branches/13/res/res_stasis.c <https://reviewboard.asterisk.org/r/4519/#comment25382> +1 to this. I think following the pattern for other applications that do this is good - they just do success/failed. Just a comment on the code review itself: Please always fill in the "Testing Done" field with the testing you've done. - Joshua Colp On March 23, 2015, 4:36 a.m., Ashley Sanders wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4519/ > ----------------------------------------------------------- > > (Updated March 23, 2015, 4:36 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24802 > https://issues.asterisk.org/jira/browse/ASTERISK-24802 > > > Repository: Asterisk > > > Description > ------- > > When an error occurs while writing to a web socket, the web socket is > disconnected and the event is logged. A side-effect of this, however, is that > any application on the other side waiting for a response from Stasis is left > hanging indefinitely (as there is no mechanism presently available for > notifying interested parties about web socket error states in Stasis). > > To remedy this scenario, this patch introduces a new channel variable: > STASIS_STATUS. > > The possible values for STASIS_STATUS are: > INITIALIZING - Indicates Stasis is starting > ACTIVE - The channel is active in Stasis > SUCCESS - The channel has exited Stasis without any failures > FAILED - Something caused Stasis to croak. Some (not all) possible > reasons for this: > - The app registry is not instantiated; > - The app requested is not registered; > - The app requested is not active; > - Stasis couldn't send a start message > > ***Note*** This is just the patch to the Asterisk source. The testsuite > review is coming soon to a reviewboard near you (well, this reviewboard.) > https://reviewboard.asterisk.org/r/4520 > > > Diffs > ----- > > ./branches/13/res/res_stasis.c 433290 > ./branches/13/apps/app_stasis.c 433290 > > Diff: https://reviewboard.asterisk.org/r/4519/diff/ > > > Testing > ------- > > > Thanks, > > Ashley Sanders > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev