----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3070/ -----------------------------------------------------------
(Updated Dec. 16, 2013, 2:47 p.m.) Review request for Asterisk Developers, David Lee, kmoore, Mark Michelson, and rmudgett. Changes ------- Addresses a lot of review feedback. Adds an argument to the POST bridges function to allow the user to set the name of the bridge. 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 (updated) ----- /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