GEANT / CAT

CAT - the Configuration Assistant Tool for Enterprise Wi-Fi networks such as eduroam
Other
93 stars 52 forks source link

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

Open jornane opened 1 year ago

jornane commented 1 year ago

Issue type

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

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

Currently eep-generic has the following options set:

twoln commented 1 year ago

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