Skip to content

#83 silent error fix#87

Open
previder-rick wants to merge 2 commits intodevelopfrom
fixes/83-silent-failure-unsupported-eap
Open

#83 silent error fix#87
previder-rick wants to merge 2 commits intodevelopfrom
fixes/83-silent-failure-unsupported-eap

Conversation

@previder-rick
Copy link
Contributor

No description provided.

Copy link
Member

@jornane jornane left a comment

Choose a reason for hiding this comment

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

Regression: When attempting to connect a supported provider (such as eduroam Visitor Access (NL)), I get the following error:

eduroam - Exception

EAP Configuration error:

Connection of type CredentialsConnector is not supported.

// this.SetActiveContent(new ConnectViewModel(this, eapConfig, defaultConnector));
// break;
default:
MessageBox.Show(string.Format(EduRoam.Localization.Resources.ErrorUnsupportedConnectionType, connector?.GetType().Name), "eduroam - Exception", MessageBoxButton.OK, MessageBoxImage.Error);
Copy link
Member

Choose a reason for hiding this comment

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

Does this also show eduroam when using govroam?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not anymore, changed it to use the Settings.Settings.ApplicationName

Copy link
Member

@jornane jornane left a comment

Choose a reason for hiding this comment

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

The error message has eduroam - Exception as title, this is incorrect for govroam.

@previder-rick previder-rick requested a review from jornane December 1, 2025 15:44
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