----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3070/#review10427 -----------------------------------------------------------
/branches/12/main/bridge.c <https://reviewboard.asterisk.org/r/3070/#comment19883> Delete this as it is already set by the struct initializer. /branches/12/main/bridge.c <https://reviewboard.asterisk.org/r/3070/#comment19884> Stray blank line. - rmudgett On Dec. 16, 2013, 8:47 p.m., Jonathan Rose wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3070/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2013, 8:47 p.m.) > > > Review request for Asterisk Developers, David Lee, kmoore, Mark Michelson, > and rmudgett. > > > Repository: Asterisk > > > Description > ------- > > So there has been at least a little bit of clammoring internally about having > the need to be able to tell when a bridge belongs to a specific parking lot > or app_confbridge conference. This patch provides two new properties to the > bridge. > > BridgeCreator - Provides the name of a system which is responsible for > creating the bridge. This includes things such as 'AgentPool', 'Parking', > 'ConfBridge', 'Stasis', and other systems which can create bridges. > > BridgeName - Provides the name given to the bridge to refer to it internally > by the creator. > > BridgeCreator may be set or it may be unset (in which case it will appear as > a zero length string). BridgeName will only appear when BridgeCreator is > also set, but it is also optional. So you have the following possibilities: > > neither BridgeCreator nor BridgeName > BridgeCreator but not BridgeName > BridgeCreator and BridgeName > > > Diffs > ----- > > /branches/12/rest-api/api-docs/bridges.json 403725 > /branches/12/res/res_stasis.c 403725 > /branches/12/res/res_ari_bridges.c 403725 > /branches/12/res/parking/parking_bridge.c 403725 > /branches/12/res/ari/resource_bridges.c 403725 > /branches/12/res/ari/resource_bridges.h 403725 > /branches/12/res/ari/ari_model_validators.c 403725 > /branches/12/res/ari/ari_model_validators.h 403725 > /branches/12/main/stasis_bridges.c 403725 > /branches/12/main/manager_bridges.c 403725 > /branches/12/main/bridge_basic.c 403725 > /branches/12/main/bridge.c 403725 > /branches/12/include/asterisk/stasis_bridges.h 403725 > /branches/12/include/asterisk/stasis_app.h 403725 > /branches/12/include/asterisk/bridge_internal.h 403725 > /branches/12/include/asterisk/bridge.h 403725 > /branches/12/doc/appdocsxml.xslt 403725 > /branches/12/apps/app_confbridge.c 403725 > /branches/12/apps/app_bridgewait.c 403725 > /branches/12/apps/app_agent_pool.c 403725 > > Diff: https://reviewboard.asterisk.org/r/3070/diff/ > > > Testing > ------- > > Checked output for Manager events displaying Bridge snapshots > > Event: BridgeEnter > Privilege: call,all > BridgeUniqueid: a72735f6-04ac-499b-a2da-bb997d6f99ab > BridgeType: parking > BridgeTechnology: holding_bridge > BridgeCreator: Parking > BridgeName: default > BridgeNumChannels: 1 > Channel: PJSIP/pjgold-00000000 > ChannelState: 6 > ChannelStateDesc: Up > CallerIDNum: pjgold > CallerIDName: pjgold > ConnectedLineNum: <unknown> > ConnectedLineName: <unknown> > AccountCode: > Context: default > Exten: 700 > Priority: 1 > Uniqueid: 1386949677.0 > > > Checked output of GET /bridges on ARI in petstore: > > [ > { > "id": "a72735f6-04ac-499b-a2da-bb997d6f99ab", > "channels": [], > "technology": "holding_bridge", > "bridge_creator": "Parking", > "bridge_class": "parking", > "bridge_type": "holding", > "bridge_name": "default" > }, > { > "id": "80e19b8f-5a04-464a-885c-4c119764bf83", > "channels": [ > "1386949828.4" > ], > "technology": "holding_bridge", > "bridge_creator": "BridgeWait", > "bridge_class": "base", > "bridge_type": "holding", > "bridge_name": "testname" > } > ] > > > Thanks, > > Jonathan Rose > >
-- _____________________________________________________________________ -- 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