Hi Brian, On Wed, 2019-03-20 at 10:56 -0700, Gix, Brian wrote: Hi Inga, From: Stotland, Inga Hi Brian, On Tue, 2019-03-19 at 14:00 -0700, Brian Gix wrote: The added D-Bus APIs enable Applications to function in a Provisioner Initiator role, and as a Configuration Client. --- doc/mesh-api.txt | 363 ++++++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 348 insertions(+), 15 deletions(-) diff --git a/doc/mesh-api.txt b/doc/mesh-api.txt index 0b341a0f9..85d1f83d2 100644 --- a/doc/mesh-api.txt +++ b/doc/mesh-api.txt @@ -30,6 +30,7 @@ Methods: org.bluez.mesh.Error.InvalidArguments void Cancel(void) + Cancels an outstanding provisioning request initiated by Join() method. @@ -109,6 +110,37 @@ Methods: PossibleErrors: org.bluez.mesh.Error.NotFound + uint64 token CreateNetwork(object app_root, array{byte}[16] uuid) + + This is the first method that an application calls to become + a Provisioner node, and a Configuration Client on a newly + created Mesh Network. + + The app_root parameter is a D-Bus object root path of the + application that implements org.bluez.mesh.Application1 + interface, and a org.bluez.mesh.Provisioner1 interface. The + application represents a node where child mesh elements have + their own objects that implement org.bluez.mesh.Element1 + interface. The application hierarchy also contains a provision + agent object that implements org.bluez.mesh.ProvisionAgent1 + interface. The standard DBus.ObjectManager interface must be + available on the app_root path. + + The uuid parameter is a 16-byte array that contains Device UUID. + + The returned token must be preserved by the application in + order to authenticate itself to the mesh daemon and attach to + the network as a mesh node by calling Attach() method or + permanently remove the identity of the mesh node by calling + Leave() method. + + The other information the bluetooth-meshd daemon will preserve + about the initial node, is to give it the initial primary + unicast address (0x0001), and create and assign a net_key as the + primary network net_index (0x000). + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments Mesh Node Hierarchy =================== @@ -146,6 +178,30 @@ Methods: org.bluez.mesh.Error.InvalidArguments org.bluez.mesh.Error.NotFound + void SendWithDeviceKey(object element_path, uint16 destination, + uint16 net_index, array{byte} data) + + This method is used to send a message originated by a local + model encoded with the device key of the remote node. + + The element_path parameter is the object path of an element from + a collection of the application elements (see Mesh Application + Hierarchy section). + + The destination parameter contains the destination address. This + destination must be a uint16 to a unicast address, or a well + known group address. + + The net_index parameter is the subnet index of the network on + which the message is to be sent. + + The data parameter is an outgoing message to be encypted by the + meshd daemon and sent on. + + Possible errors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotFound + void Publish(object element_path, uint16 model, array{byte} data) This method is used to send a publication originated by a local @@ -224,12 +280,211 @@ Properties: This property indicates whether the periodic beaconing is enabled (true) or disabled (false). + uint8 BeaconFlags [read-only] + + This property may be read at any time to determine the flag + field setting on sent and received beacons of the primary + network key. + + uint32 IvIndex [read-only] + + This property may be read at any time to determine the IV_Index + that the current network is on. This information is only useful + for provisioning. + uint32 SecondsSinceLastHeard [read-only] This property may be read at any time to determine the number of seconds since mesh network layer traffic was last detected on this node's network. +Mesh Provisioning Hierarchy +============================ +Service org.bluez.mesh +Interface org.bluez.mesh.Provisioning1 We may want to call this interface "org.bluez.mesh.Manager1", because it contains methods surpassing provisioning procedure +Object path /org/bluez/mesh/node<xxxx> + where xxxx is a 4-digit hexadecimal number generated by daemon + +Methods: + void UnprovisionedScan(uint16 seconds) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to start listening + (scanning) for unprovisioned devices in the area. Scanning + will continue for the specified number of seconds, or, if 0 is + specified, then continuously until UnprovisionedScanCancel() is + called or if AddNode() method is called. + + Each time a unique unprovisioned beacon is heard, the + ScanResult() method on the app will be called with the result. + + PossibleErrors: + org.bluez.mesh.Error.NotAuthorized + + void UnprovisionedScanCancel(void) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to stop listening + (scanning) for unprovisioned devices in the area. + + PossibleErrors: + org.bluez.mesh.Error.NotAuthorized + + void AddNode(array{byte}[16] uuid) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to add the + unprovisioned device specified by uuid, to the Network. + + The uuid parameter is a 16-byte array that contains Device UUID + of the unprovisioned device to be added to the network. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + + void AddNetKey(uint16 net_index) I suggest we change this method to AddNetKey(uint16 dest, uint16 net_index) Since in the current proposed approach the daemon is the only entity in possession of key vaues, the configuration messages that contain the key values in their payload will need to be generated by the daemon. The modified method will allow to send the network key(s) to any node. If a network key specified by the net_index, does not exist, it will be generated and recorded, effectively creating a new subnet. Alternatively, this functionality can be covered by two methods: CreateSubnet(uint16 net_index) to generate a key value and AddNetKey(uint16 dest, uint16 net_index) to send the keys to a remote or local node So I think the biggest problem with having methods *sometimes* add a key to the *local* database, but always add the key to a remote node is Indeterminacies. We are trying to potentially do two things at once, and either one or both can fail. I strongly believe that the method calls to add a key (or delete it or update it) to our local key storage needs to be a distinct requests... Yes, we only save *one* method call by overloading the functionality into a single call. But separating Local operations from Remote operations keeps it clear, if there is a failure, at exactly which step the failure occurred. The Application is not left wondering if there is an unused key in the local database, or if the remote device now has a key that nobody else does. In order to separate "local" form "remote node" operations: Rename: AddNetKey(idx) -> CreateSubnet(idx) UpdateNetKey(idx) -> UpdateSubnet(idx) DeleteKey(idx) -> DeleteSubnet(idx) AddAppKey(net_idx, app_idx) -> CreateAppKey(net_idx, app_idx) UpdateAppKey(net_idx, app_idx) -> UpdateAppKey(app_idx) New methods to send the keys to nodes: AddNetKey(uint16 dest, uint16 net_idx, boolean update) AddAppKey(uint16 dest, uint16 app_idx, boolean update) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to generate and add a new + network subnet key. + + The net_index parameter is a 12-bit value (0x001-0xFFF) + specifying which net key to add. + + This call affects the local bluetooth-daemon key database only. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.AlreadyExists + + void UpdateNetKey(uint16 net_index) I suggest we change this method to UpdateNetKey(uint16 dest, uint16 net_index) This will allow to send an updated value of the network key to any node. If the updated network key value specified by the net_index, does not exist, it will be generated and recorded. Same comment here... I intended this for this method to modify the local key database only, with no overloaded functionality of sending the update key to one of the potentially hundreds of other nodes. So, when the network key is updated locally, shouldn't the phase change automatically? + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to generate a new network + subnet key, and set it's key refresh state to Phase 1. + + The net_index parameter is a 12-bit value (0x000-0xFFF) + specifying which net key to update. Note that the subnet must + exist prior to updating. + + This call affects the local bluetooth-daemon key database only. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.DoesNotExist + + void DeleteNetKey(uint16 net_index) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to delete a subnet. + + The net_index parameter is a 12-bit value (0x001-0xFFF) + specifying which net key to delete. The primary net key (0x000) + may not be deleted. + + This call affects the local bluetooth-daemon key database only. Just curious what would happen if there are remote nodes in possession of this network key and this method is called? Potentially this may lead to an "unconfigurable" subnet... + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.DoesNotExist + + void SetKeyPhase(uint16 net_index, uint8 phase) + This method is used to set the master key update phase of the + given subnet. + + The net_index parameter is a 12-bit value (0x000-0xFFF) + specifying which subnet phase to set. + + The phase parameter is used to cycle the local key database + through the phases as defined by the Mesh Profile Specification. + Allowed values: + 0 - Cancel Key Refresh (May only be called from Phase 1, + and should never be called once the new key has + started propagating) + 1 - Invalid Argument (see NetKeyUpdate method) + 2 - Go to Phase 2 (May only be called from Phase 1) + 3 - Complete Key Refresh procedure (May only be called + from Phase 2) + + This call affects the local bluetooth-daemon key database only. This method is also intended to modify the state of the keys locally only. Only after the local phase has been recorded would the phase change be propogated to the wider mesh + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.DoesNotExist + + void AddAppKey(uint16 net_index, uint16 app_index) Similar to the network key methods, needs to include dest parameter + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to generate and add a new + application key. + + The net_index parameter is a 12-bit value (0x000-0xFFF) + specifying which net key to bind the application key to. + + The app_index parameter is a 12-bit value (0x000-0xFFF) + specifying which app key to add. + + This call affects the local bluetooth-daemon key database only. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.AlreadyExists + + void UpdateAppKey(uint16 app_index) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to generate a new + application key. + + The app_index parameter is a 12-bit value (0x000-0xFFF) + specifying which app key to update. Note that the subnet that + the key is bound to must exist and be in Phase 1. + + This call affects the local bluetooth-daemon key database only. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.DoesNotExist + + void DeleteAppKey(uint16 app_index) + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to delete an application + key. + + The app_index parameter is a 12-bit value (0x000-0xFFF) + specifying which app key to delete. + + This call affects the local bluetooth-daemon key database only. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.DoesNotExist + + void DeleteRemoteNode(uint16 primary, uint8 cnt) Let's call this method DeleteNode() Do we need to specify the element count? I want to be clear that this again only affects the local key database. We do have the ability to delete Local nodes with the Leave() method. I want to be clear with this method that again, no communication is made... This simply deletes a Remote Node Device key from the local database. I included a count parameter so that storage of the device keys could be optimized such that all element unicast addresses are accounted for. It is 100% legal to send a Device Key encoded message to a non-primary element of a Node, so we might have redundant storages of the device keys under every unicast address, in the local database. + + This method is used by the application that supports + org.bluez.mesh.Provisioner1 interface to delete a remote node + from the local device key database. + + The primary parameter specifies the unicast address of the + the node being deleted, and the number of elements that were + assigned to it. + + This call affects the local bluetooth-daemon key database only. + + PossibleErrors: + org.bluez.mesh.Error.InvalidArguments + org.bluez.mesh.Error.NotAuthorized + org.bluez.mesh.Error.DoesNotExist Mesh Application Hierarchy ========================== @@ -247,6 +502,7 @@ An example mesh application hierarchy may look like this: | - org.freedesktop.DBus.ObjectManager | - org.bluez.mesh.Application1 | - org.bluez.mesh.Attention1 (optional) + | - org.bluez.mesh.Provisioner1 (optional,Provisioner) | -> /com/example/agent | | - org.bluez.mesh.ProvisionAgent1 @@ -325,6 +581,22 @@ Methods: The data parameter is the incoming message. + void DevKeyMessageReceived(uint16 source, uint16 net_index, + array{byte} data) + + This method is called by meshd daemon when a message arrives + addressed to the application, which was sent with the remote + node's device key. + + The source parameter is unicast address of the remote + node-element that sent the message. + + The net_index parameter indicates what subnet the message was + received on, and if a response is required, the same subnet + must be used to send the response. + + The data parameter is the incoming message. + void UpdateModelConfiguration(uint16 model_id, dict config) This method is called by bluetooth-meshd daemon when a model's @@ -383,7 +655,7 @@ Object path freely definable This is an optional interface that implements health attention timer. Methods: - void SetTimer(uint8 element_index, uint16 time) + void SetTimer(uint8 element_index, uint16 time) The element_index parameter is the element's index within the node where the health server model is hosted. @@ -406,6 +678,76 @@ Methods: org.bluez.mesh.Error.NotSupported +Mesh Provisioner Hierarchy +============================ +Service unique name +Interface org.bluez.mesh.Provisioner1 +Object path freely definable + + ScanResult(int8 rssi, array{byte} data) + + The method is called from the bluetooth-meshd daemon when a + unique UUID has been seen during UnprovisionedScan() for + unprovsioned devices. + + The rssi parameter is a signed, normalized measurement of the + signal strength of the recieved unprovisioned beacon. + + The data parameter is a variable length byte array, that may + have 1, 2 or 3 distinct fields contained in it including the 16 + byte remote device UUID (always), a 32 bit mask of OOB + authentication flags (optional), and a 32 bit URI hash (if URI + bit set in OOB mask). Whether these fields exist or not is a + decision of the remote device. + + If a beacon with a UUID that has already been reported is + recieved by the daemon, it will be silently discarded unless it + was recieved at a higher rssi power level. + + + uint16 net_index, uint8 flags, uint32 iv_index, uint16 unicast + RequestProvData () + + This method is implemented by a Provisioner capable application + and is called when the remote device has been fully + authenticated and confirmed. + + Return Parameters are from the Mesh Profile Spec: + net_index - Subnet index of the net_key + flags - Flags for IV_Update and Key Refresh + iv_index - Current IvIndex being used on the network + unicast - Primary Unicast address of the new node + + PossibleErrors: + org.bluez.mesh.Error.Abort + + void AddNodeComplete(array{byte}[16] uuid, uint16 unicast, uint8 count) + + This method is called when the node provisioning initiated + by an AddNode() method call successfully completed. + + The unicast parameter is the primary address that has been + assigned to the new node, and the address of it's config server. + + The count parameter is the number of unicast addresses assigned + to the new node. + + The new node may now be sent messages using the credentials + supplied by the RequestProvData method. + + void AddNodeFailed(array{byte}[16] uuid, string reason) + + This method is called when the node provisioning initiated by + AddNode() has failed. Depending on how far Provisioning + proceeded before failing, some cleanup of cached data may be + required. + + The reason parameter identifies the reason for provisioning + failure. The defined values are: "aborted", "timeout", + "bad-pdu", "confirmation-failed", "out-of-resources", + "decryption-error", "unexpected-error", + "cannot-assign-addresses". + Provisioning Agent Hierarchy ============================ Service unique name @@ -441,6 +783,7 @@ Methods: the local role is Provisioner. void DisplayString(string value) + This method is called when the Daemon has something important for the Agent to Display, but does not require any additional input locally. For instance: "Enter "ABCDE" on remote device". @@ -451,9 +794,8 @@ Methods: for the Agent to Display, but does not require any additional input locally. For instance: "Enter 14939264 on remote device". - The type parameter indicates the display method. Allowed values + The type parameter indicates the display method. Allowed values are: - "blink" - Locally blink LED "beep" - Locally make a noise "vibrate" - Locally vibrate @@ -466,12 +808,11 @@ Methods: uint32 PromptNumeric(string type) - This method is called when the Daemon has requires the user to - enter a 1-99999999 digit decimal value. + This method is called when the Daemon requests the user to + enter a decimal value between 1-99999999. The type parameter indicates the input method. Allowed values are: - "blink" - Enter times remote LED blinked "beep" - Enter times remote device beeped "vibrate" - Enter times remote device vibrated @@ - 483,14 +824,13 @@ Methods: This agent should prompt the user for specific input. For instance: "Enter value being displayed by remote device". - array{byte} PromptStatic(string type) + array{byte}[16] PromptStatic(string type) This method is called when the Daemon requires a 16 octet byte array, as an Out-of-Band authentication. The type parameter indicates the input method. Allowed values are: - "static-oob" - return 16 octet array "in-alpha" - return 16 octet alpha array @@ -511,7 +851,6 @@ Properties: array{string} Capabilities [read-only] An array of strings with the following allowed values: - "blink" "beep" "vibrate" @@ -528,7 +867,6 @@ Properties: Indicates availability of OOB data. An array of strings with the following allowed values: - "other" "uri" "machine-code-2d" @@ -546,8 +884,3 @@ Properties: Uniform Resource Identifier points to out-of-band (OOB) information (e.g., a public key) - - -Mesh Provisioner Hierarchy -========================== -<TBD> Inga