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

Difference between eap-generic and eap-config-full is still confusing for admins #279

Open
1 of 4 tasks
jornane opened this issue Jul 17, 2023 · 4 comments
Open
1 of 4 tasks

Comments

@jornane
Copy link
Contributor

jornane commented Jul 17, 2023

Issue type

  • Questions about the eduroamCAT or its usage should be sent to the users mailing list.

  • Defect - Crash/memory corruption.

  • Defect - Non-compliance with a standards document or incorrect OS API usage.

  • Defect - Unexpected behaviour (obvious or has been verified by a project member).

  • New feature request.

Defect/Feature description

How to reproduce issue

  1. Go to the admin portal, go to the "Installer Fine-Tuning and Download" page for a PEAP or TTLS profile, that has an RCOI set.
  2. Observe there is an EAP config device there.
  3. Click the Download eduroam® Installer button.
  4. Try to load the resulting .eap-config file in geteduroam.
  5. Observe that Passpoint is not configured.

Detail of issue

This is the same issue as #271, which was closed without fully resolving it, and which is not replied to anymore. In #271, the "EAP config" profile was removed from the user view, but it is still in the admin view.

Administrators know that geteduroam uses eap-config profiles under the hood, so in order to test their CAT profiles, they download the .eap-config file from CAT and try to load it in the app, which is a supported feature of the geteduroam app on all platforms.

Recommended solution

  • Add a device to the "Installer Fine-Tuning and Download" list, geteduroam, which is eap-generic.
  • Possibly remove the "EAP config" (eap-config-full) device type from the admin interface as well; the format is not supported by any current application, and as such is of little use to admins.

Possibly add a device type which is specificly for geteduroam, in the same fashion as the device types android_4_7 and android_recent. In fact, android_recent refers to Android 8 and up, which is also supported by geteduroam.

@restena-sw
Copy link
Contributor

Looking at the tickbox, it's "Defect - Unexpected Bahviour"? I don't see it that way really.

The current download produces a schema-valid file - maybe geteduroam has issues parsing that correctly; but then the current /name/ also doesn't suggest it should be used with geteduroam.

The same download section has a download for "Android 8 and higher", and my naive expectation is that an admin wanting an Android 8+ download uses that, rather than associating heavily on something they've heard somewhere and jumping to wrong conclusions.

This is something I understand more from an unsuspecting end-user side, not so much from an admin.

So, it's "unexpected behaviour" just fine, just not code-side but consumer-side. :-)

We can do your recommendation #1 easily; it's essentially cloning "Android 8 and higher" simply with a different display string "geteduroam" associated with it. If that makes the world a better place, why not.

@twoln
Copy link
Contributor

twoln commented Jul 18, 2023

Absolutely, and actually this probably comment does not belong to GitHub at all. This is not a coding but purely configuration issue. When preparing the new release for production I will ask Jorn to check that things look and work as he would expect.

@twoln
Copy link
Contributor

twoln commented Jul 18, 2023

Currently eep-generic has the following options set:

  • compatibility for eduroamCAT - probably should be removed
  • single language - probably correct
  • OpenRoaming support - probably to be removed
    The title for this button should then be either "geteduroam" or "geteduroam profile". The second one would require a translation, but since this is admin area we can just let it go. I would be against just "geteduroam" as this could be misleading as pointing to the app itself.
    What about the icon for this - I guess the geteduroam one would be just fine?
    Finally - should we wait for the new release - I would thing that this is reasonable - people expect changes in such moments.

@twoln
Copy link
Contributor

twoln commented Jul 31, 2023

I have removed eep-config-full from the admin download as well. But I will have more requests for clarification later.

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

No branches or pull requests

3 participants