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

feat: Hide WPA3 feature based on Kura property #5607

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

pierantoniomerlino
Copy link
Contributor

@pierantoniomerlino pierantoniomerlino commented Dec 6, 2024

This PR adds a way to disable the new WPA3 feature based on a specific Kura property.

Related Issue: This PR fixes/closes N/A

Description of the solution adopted: The #5586 added the (experimental) support for WPA3 Wifi Security. This PR adds a Kura property (kura.wpa3.wifi.security.enable) that hides/shows the WPA3 entries on the Wireless tab of the webUI. Moreover, the same property inhibits/allows the application of a network configuration containing WPA3 related properties to the framework.
The default behavior is to hide the feature.

@pierantoniomerlino pierantoniomerlino force-pushed the wpa3_capability_hide_property branch from f35da42 to b59ea8a Compare December 6, 2024 11:30
@pierantoniomerlino pierantoniomerlino changed the title feat: Wpa3 capability hide property feat: Hide WPA3 feature based on Kura property Dec 6, 2024
@pierantoniomerlino pierantoniomerlino marked this pull request as ready for review December 6, 2024 11:35
@@ -152,6 +158,7 @@ interface TabWirelessUiUiBinder extends UiBinder<Widget, TabWirelessUi> {
private final NetworkTabsUi netTabs;
private final ListDataProvider<GwtWifiHotspotEntry> ssidDataProvider = new ListDataProvider<>();
private final SingleSelectionModel<GwtWifiHotspotEntry> ssidSelectionModel = new SingleSelectionModel<>();
// private boolean isWPA3WifiSecuritySupported;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this commented code

@@ -375,4 +375,10 @@ public void shouldGetDefaultLogManagerProperty() {
assertFalse(systemService.getDefaultLogManager().isPresent());
}

@TestTarget(targetPlatforms = { TestTarget.PLATFORM_ALL })
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update copyright header

@@ -195,9 +195,9 @@ public ArrayList<GwtGroupedNVPair> findSystemProperties(GwtXSRFToken xsrfToken)
// kura properties
SystemService systemService = ServiceLocator.getInstance().getService(SystemService.class);
Properties kuraProps = systemService.getProperties();
SortedSet kuraKeys = new TreeSet(kuraProps.keySet());
SortedSet kuraKeys = new TreeSet(kuraProps.stringPropertyNames());
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update copyright header

@@ -25,4 +25,5 @@
<reference bind="setDnsServerService" cardinality="1..1" interface="org.eclipse.kura.internal.linux.net.dns.DnsServerService" name="DNSService" policy="static" />
<reference name="CryptoService" interface="org.eclipse.kura.crypto.CryptoService" bind="setCryptoService" unbind="unsetCryptoService" cardinality="1..1" policy="static"/>
<reference name="KeystoreService" interface="org.eclipse.kura.security.keystore.KeystoreService" bind="setKeystoreService" unbind="unsetKeystoreService" cardinality="0..n" policy="dynamic"/>
<reference bind="setSystemService" cardinality="1..1" interface="org.eclipse.kura.system.SystemService" name="SystemService" policy="static"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update copyright header

@pierantoniomerlino pierantoniomerlino force-pushed the wpa3_capability_hide_property branch from 547db70 to 78ff6aa Compare December 11, 2024 15:27
@nicolatimeus
Copy link
Contributor

Should we also filter away/prohibit connection to the networks that use WPA3 from the scan results in web ui when in station mode?

@pierantoniomerlino
Copy link
Contributor Author

@nicolatimeus I was convinced to have already add that filter... but it seems that I lost that modification in some rebase. I'll add it.

Copy link
Contributor

@nicolatimeus nicolatimeus left a comment

Choose a reason for hiding this comment

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

Tested as working, build is failing

@MMaiero MMaiero force-pushed the wpa3_capability_hide_property branch from 107760b to acee329 Compare December 16, 2024 20:34
@pierantoniomerlino pierantoniomerlino merged commit e060bbe into develop Dec 17, 2024
4 checks passed
@pierantoniomerlino pierantoniomerlino deleted the wpa3_capability_hide_property branch December 17, 2024 08:01
MMaiero pushed a commit that referenced this pull request Dec 20, 2024
* Hide WPA3 feature based on property

Signed-off-by: pierantoniomerlino <[email protected]>

* Updated copyright

Signed-off-by: pierantoniomerlino <[email protected]>

* Removed commented line; updated copyright headers

Signed-off-by: pierantoniomerlino <[email protected]>

* Added filtering of WPA3 on hotspot scans

Signed-off-by: pierantoniomerlino <[email protected]>

---------

Signed-off-by: pierantoniomerlino <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants