----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3402/#review11427 -----------------------------------------------------------
Ship it! While I'm not a fan of putting in changes that fix problems whose origins are mysterious, the actual changes you're making here should not cause harm, and they're fixing an observed problem. Go for it! - Mark Michelson On March 28, 2014, 2:19 p.m., Scott Griepentrog wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3402/ > ----------------------------------------------------------- > > (Updated March 28, 2014, 2:19 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23548 > https://issues.asterisk.org/jira/browse/ASTERISK-23548 > > > Repository: Asterisk > > > Description > ------- > > When running asterisk in a virtual machine, responses to ARI requests would > frequently be missing. A race condition related to closing the socket > immediately after writing data onto it is resolved in this patch by insuring > the output stream is flushed, and then informing TCP of the shutdown prior to > the close. > > > Diffs > ----- > > /branches/1.8/main/tcptls.c 411312 > /branches/1.8/main/manager.c 411312 > /branches/1.8/main/http.c 411312 > > Diff: https://reviewboard.asterisk.org/r/3402/diff/ > > > Testing > ------- > > Tested on Ubuntu server 12.04 with Sam's json api test from issue. > > > Thanks, > > Scott Griepentrog > >
-- _____________________________________________________________________ -- 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
