Might be worth refactoring this function to take array of device IDs &[&str] and send messages in a for loop, to avoid creating tunnelbroker_client for each device when called this in a loop

bartek mentioned this in D12378: [Identity] send `IdentityDeviceListUpdated` after logout.Jun 12 2024, 8:50 AM

refactor to use array od device IDs

In D12377#352213, @bartek wrote:

Might be worth refactoring this function to take array of device IDs &[&str] and send messages in a for loop, to avoid creating tunnelbroker_client for each device when called this in a loop

Good call

Harbormaster completed remote builds in B29664: Diff 41288.Jun 13 2024, 8:55 AM

Closed by commit rCOMMa3d5a4e16790: [Identity] implement sending `IdentityDeviceListUpdated` using Tunnelbroker… (authored by kamil). · Explain WhyJun 13 2024, 9:08 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMa3d5a4e16790: [Identity] implement sending `IdentityDeviceListUpdated` using Tunnelbroker….

kamil mentioned this in D12438: [terraform] Deploy Identity `0.23-staging` to staging.Jun 14 2024, 2:30 AM

kamil mentioned this in rCOMM47f0ef750274: [terraform] Deploy Identity `0.23-staging` to staging.Jun 14 2024, 2:55 AM