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

Allow setting autheap for phase2 #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ghalse
Copy link
Contributor

@ghalse ghalse commented Jul 19, 2021

In order to test some phase2 types, we need to use autheap= rather than auth= in the eapol_test config.

An example of where this is necessary is with EAP-TTLS/EAP-GTC or EAP-TTLS/EAP-MSCHAPv2 (as opposed to EAP-TTLS/MSCHAPv2). Our use case for it is testing GTC passwords.

This patch allows the phase2 type to explicitly specify that autheap= is required by prefixing the EAP type with EAP-. In other words, -2 MSCHAPV2 will result in auth=MSCHAPV2 but -2 EAP-MSCHAPV2 will result in autheap=MSCHAPV2.

@spaetow
Copy link

spaetow commented Sep 10, 2021

I second this. It would resolve #13.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants