On Tue, Nov 29, 2016 at 4:34 AM, Joshua Colp <jc...@digium.com> wrote:
> On Sun, Nov 27, 2016, at 11:29 PM, Corey Farrell wrote: > > A review [1] has been posted to fix an issue where TLS servers would > > not be restarted unless the bind address was changed. This would > > prevent use of new certificates if available. Unfortunately this > > change does cause an ABI change. Fields are added to public > > structures 'struct ast_tls_config' and 'struct > > ast_tcptls_session_args'. Within Asterisk itself these structures are > > used by app_externalivr, chan_sip, res_http_websocket, http.c and > > manager.c. > > > > tcptls.h does not provide an allocation method for it's structures. > > These means it is impossible to add fields to these structures without > > breaking the ABI. How does everyone feel about moving forward with > > the fix as is? > > When it comes to ABI compatibility I take what exactly is being changed > into account. In the case of the TCP/TLS code it's not something I'd see > outside code or developers using (the commercial modules certainly don't > use it) which is why I'm personally not opposed to accepting it. > > Any other thoughts? Do we want to be strict and only allow on master? > I'd say it's OK for 13. I can't think of a scenario where an external module would be using these APIs.
-- _____________________________________________________________________ -- 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