Where do you suspect the issue? Issue in mesh library, e.g. packet

<a target="_blank" rel="noopener noreferrer" href="https://private-user-images.githubu

There will be a timeout. The timeout is set using: <div class="Box Box--conden

Send message await forever about ios-nrf-mesh-library HOT 11 OPEN

jack8525 commented on July 28, 2024 1

Send message await forever

from ios-nrf-mesh-library.

Comments (11)

philips77 commented on July 28, 2024

Thank you for reporting the issue. I'll look into it and come back to you.

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

For Acknowledged messages there's a timer on Access Layer that should try to retransmit the message and time out eventually.

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

OK. I see where the issue may occur.

AccessLayer sets the repeat & timeout timer only for acknowledge messages sent to a Unicast Address:

IOS-nRF-Mesh-Library/nRFMeshProvision/Layers/Access Layer/AccessLayer.swift

Lines 239 to 244 in 293210d

 // Set timers for the acknowledged messages. 

 // Acknowledged messages sent to a Group address won't await a Status. 

 if message is AcknowledgedMeshMessage, 

 destination.address.isUnicast { 

 createReliableContext(for: pdu, sentFrom: element, withTtl: initialTtl, using: keySet) 

 }

That means, that acknowledged messages sent to a Group Address (or Virtual Address) don't get that timer set. Such messages may be sent to 0+ nodes and the transmitter can receive 0+ responses. In that case, it is the underlying layer that should notify the network manager, just like for unacknowledged messages.

However, as you pointed out, when the Acknowledged Mesh Message is unsegmented, like ConfigOnOffSet, and sending the message fails, the network manager is never notified:

IOS-nRF-Mesh-Library/nRFMeshProvision/Layers/Lower Transport Layer/LowerTransportLayer.swift

Lines 257 to 260 in 293210d

 if !pdu.message!.isAcknowledged { 

 networkManager.notifyAbout(error: error, duringSendingMessage: pdu.message!, 

 from: localElement, to: pdu.destination) 

 }

I'll prepare a fix.

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

Btw, for multicast addresses it is recommended not to use Acknowledged messages, but their Unacknowledged counterparts. This significantly decreases the traffic in the network.

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

Hmm... I did some more digging and seems like the Network Layer doesn't throw bearerDidClose when the message is sent to a group address, as such message may be handled internally:

IOS-nRF-Mesh-Library/nRFMeshProvision/Layers/Network Layer/NetworkLayer.swift

Lines 156 to 167 in 293210d

 // Loopback interface. 

 if shouldLoopback(networkPdu) { 

 handle(incomingPdu: networkPdu.pdu, ofType: type) 

 // Messages sent with TTL = 1 will only be sent locally. 

 guard ttl != 1 else { return } 

 if isLocalUnicastAddress(networkPdu.destination) { 

 // No need to send messages targeting local Unicast Addresses. 

 return 

 } 

 // If the message was sent locally, don't report Bearer closer error. 

 try? transmitter.send(networkPdu.pdu, ofType: type) 

 } else {

This is the shouldLoopback method:

IOS-nRF-Mesh-Library/nRFMeshProvision/Layers/Network Layer/NetworkLayer.swift

Lines 444 to 450 in 293210d

 /// Returns whether the PDU should loop back for local processing. 

 /// 

 /// - parameter networkPdu: The PDU to check. 

 func shouldLoopback(_ networkPdu: NetworkPdu) -> Bool { 

 let address = networkPdu.destination 

 return address.isGroup || address.isVirtual || isLocalUnicastAddress(address) 

 }

As you're getting that error, that means, that you're sending the message to a Unicast Address. In that case, the network manager should wait until the response is received from the target, which happens on Access Layer.
I'll leave #600 open, but don't think it's required.

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

Could you say what message are you trying to send exactly, what are your network parameters (default?) and what is the destination address of the message?

from ios-nrf-mesh-library.

jack8525 commented on July 28, 2024

Thank you for your help. I just send some GenericOnOffSet to a Unicast Address when not connected.In this case, a timeout will not be thrown?

from ios-nrf-mesh-library.

philips77 commented on July 28, 2024

There will be a timeout. The timeout is set using:

IOS-nRF-Mesh-Library/nRFMeshProvision/Layers/NetworkParameters.swift

Lines 219 to 230 in 293210d

 /// Sets the timeout for receiving a response to an acknowledged access message. 

 /// 

 /// The ``MeshNetworkDelegate/meshNetworkManager(_:failedToSendMessage:from:to:error:)-7iylf`` 

 /// callback will be called when the response is not received before the timeout expires.. 

 /// 

 /// - parameter timeout: The timeout after which the ``AccessError/timeout`` 

 /// is reported. This shall be set to a minimum of 30 seconds, 

 /// which is also the default value. 

 /// - seeAlso: ``NetworkParameters/acknowledgmentMessageTimeout`` 

 public func discardAcknowledgedMessages(after timeout: TimeInterval) { 

 networkParameters.acknowledgmentMessageTimeout = timeout 

 }

Documentatuion

from ios-nrf-mesh-library.

jack8525 commented on July 28, 2024

I found failedToSendMessage is no callback. Because Address change to MeshMessage.
After fix it.
I can handle the timeout here.

from ios-nrf-mesh-library.

jack8525 commented on July 28, 2024

Can I handle deliveryCallbacks here?

from ios-nrf-mesh-library.

Send message await forever about ios-nrf-mesh-library HOT 11 OPEN

Comments (11)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	// Set timers for the acknowledged messages.
	// Acknowledged messages sent to a Group address won't await a Status.
	if message is AcknowledgedMeshMessage,
	destination.address.isUnicast {
	createReliableContext(for: pdu, sentFrom: element, withTtl: initialTtl, using: keySet)
	}

	if !pdu.message!.isAcknowledged {
	networkManager.notifyAbout(error: error, duringSendingMessage: pdu.message!,
	from: localElement, to: pdu.destination)
	}

	// Loopback interface.
	if shouldLoopback(networkPdu) {
	handle(incomingPdu: networkPdu.pdu, ofType: type)
	// Messages sent with TTL = 1 will only be sent locally.
	guard ttl != 1 else { return }
	if isLocalUnicastAddress(networkPdu.destination) {
	// No need to send messages targeting local Unicast Addresses.
	return
	}
	// If the message was sent locally, don't report Bearer closer error.
	try? transmitter.send(networkPdu.pdu, ofType: type)
	} else {

	/// Returns whether the PDU should loop back for local processing.
	///
	/// - parameter networkPdu: The PDU to check.
	func shouldLoopback(_ networkPdu: NetworkPdu) -> Bool {
	let address = networkPdu.destination
	return address.isGroup \|\| address.isVirtual \|\| isLocalUnicastAddress(address)
	}

	/// Sets the timeout for receiving a response to an acknowledged access message.
	///
	/// The ``MeshNetworkDelegate/meshNetworkManager(_:failedToSendMessage:from:to:error:)-7iylf``
	/// callback will be called when the response is not received before the timeout expires..
	///
	/// - parameter timeout: The timeout after which the ``AccessError/timeout``
	/// is reported. This shall be set to a minimum of 30 seconds,
	/// which is also the default value.
	/// - seeAlso: ``NetworkParameters/acknowledgmentMessageTimeout``
	public func discardAcknowledgedMessages(after timeout: TimeInterval) {
	networkParameters.acknowledgmentMessageTimeout = timeout
	}