-
Notifications
You must be signed in to change notification settings - Fork 206
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
auth: add support for LEAP and EAP-PWD (LP: #2038811) #415
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much! I think this should fix the case for the "leap" and "pwd" values of NetworkManager's 801-1x.eap
setting. But there might be more, similar cases and we should think about implementing some generic (future proof) fallback, using networkmanager.passthrough
.
9f59402
to
a9e021e
Compare
Something is still missing, the commands below are still failing:
|
66a65eb
to
62a2d52
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with the reduced scope. Let's fix the bug at hand first and work on the generic solution afterwards.
nitpick: May I just suggest to add simple testcases for the new "leap"/"pwd" EAP methods in test_keyfile.py
?
diff --git a/tests/parser/test_keyfile.py b/tests/parser/test_keyfile.py
index dbb992ed..7a6bd2ca 100644
--- a/tests/parser/test_keyfile.py
+++ b/tests/parser/test_keyfile.py
@@ -526,6 +526,12 @@ dns-search='''.format(UUID, method))
def test_keyfile_type_wifi_eap_ttls(self):
self._template_keyfile_type_wifi_eap('ttls')
+ def test_keyfile_type_wifi_eap_leap(self):
+ self._template_keyfile_type_wifi_eap('leap')
+
+ def test_keyfile_type_wifi_eap_pwd(self):
+ self._template_keyfile_type_wifi_eap('pwd')
+
def test_keyfile_type_wifi_eap_leap(self):
self.generate_from_keyfile('''[connection]
type=wifi
This changes add support for the LEAP and PWD EAP authentication methods for NetworkManager, wpa_supplicant (networkd) and the keyfile parser. Using these method through the NM GUI is causing problems because libnetplan will generate a broken keyfile. See LP: #2038811 Note that the NM GUI supports more methods not recognized by netplan and we probably should also implement them later.
62a2d52
to
e4bfabc
Compare
This changes add support for the LEAP and PWD EAP authentication methods for NetworkManager, wpa_supplicant (networkd) and the keyfile parser.
Using these method through the NM GUI is causing problems because libnetplan will generate a broken keyfile. See LP: #2038811
Note that the NM GUI supports more methods not recognized by netplan and we probably should also implement them later.
Reproducers
Description
Checklist
make check
successfully.make check-coverage
).