Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[libconnman-qt] Support VPN association state. JB#59447 #12

Merged
merged 2 commits into from Dec 19, 2022

Conversation

LaakkonenJussi
Copy link
Contributor

Add support for the association state for VPNs. This is the state when VPN is querying VPN agent (user) for credentials. Behavior is similar as with other services as association is the generic state in connmand that is the state prior to configuration.

@LaakkonenJussi
Copy link
Contributor Author

See also sailfishos/connman#34

Copy link
Member

@llewelld llewelld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all looks good to me, and in practice I found it worked really nicely when combined with the other changes.

Add support for the association state for VPNs. This is the state when
VPN is querying VPN agent (user) for credentials. Behavior is similar as
with other services as association is the generic state in connmand that
is the state prior to configuration.
Update ConnMan requirement to git190 that has the association state
change for VPNs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants