Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed Changes
This adds a function that wraps the GetSecrets method.
Draft status
It is a draft because I'm still unsure on two aspects:
{s: v}
into a Rust struct (probably not, assuming that this is where faster-than-API extension in NetworkManager happens)?Connection
struct) be exposed as an explicit Thing?This would eliminate what is currently a tiny race condition if the network is changed while the SSID/password is being queried: for then, there might be the SSID of the one and the secrets of the other being returned.