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

[Prototype] Introduce MAUI support for the client stack #1480

Closed
wants to merge 1 commit into from

Conversation

kevinchalet
Copy link
Member

This PR adds a prototype of a MAUI integration for the OpenIddict client (currently, WinUI-only).

@lnaie
Copy link

lnaie commented Jul 9, 2022

For local server I get this error, after a while: 'IDX20803: Unable to obtain configuration from: 'https://localhost:44395/.well-known/openid-configuration'.'

I ran the MAUI project in VS 2022 preview, but not as admin.

@lnaie
Copy link

lnaie commented Jul 9, 2022

Could the code for Android and iOS be published here or in some other branch?

@kevinchalet
Copy link
Member Author

For local server I get this error, after a while: 'IDX20803: Unable to obtain configuration from: 'https://localhost:44395/.well-known/openid-configuration'.'

Make sure you started the OpenIddict.Sandbox.AspNetCore.Server project, that contains the server used by the MAUI app.

Could the code for Android and iOS be published here or in some other branch?

Only WinUI is suppported ATM (the project stops building as soon as I add an Android TFM due to the fact the tooling doesn't support diacritics in paths and I don't have an Apple dev' account to try implementing iOS support).

@kevinchalet kevinchalet removed this from the 4.0.0-preview2 milestone Jul 19, 2022
@kevinchalet
Copy link
Member Author

Closing for now due to a lack of interest from the community.

@kevinchalet kevinchalet deleted the maui branch July 22, 2022 08:44
@kevinchalet kevinchalet restored the maui branch July 26, 2022 13:55
@kevinchalet kevinchalet deleted the maui branch November 26, 2022 18:47
@JohnHunterACS
Copy link

If only I'd see this branch last month. It's exactly what I'm looking for :(

@kevinchalet
Copy link
Member Author

@JohnHunterACS unfortunately, the interest was too low to justify maintaining a MAUI integration.

Not to mention that MAUI isn't exactly getting the reception such an ambitious should get (there isn't a single week without something like https://www.reddit.com/r/dotnet/comments/10g734a/mauis_potential_hindered_by_inadequate/), which makes investing on it a bit... risky. We know what Microsoft does when things don't go the way they wanted to... 😅

FWIW, I tried to contact someone from the MAUI team to see if they could be interested in seeing a universal OIDC client for MAUI but I got no reply... so 🤷🏻

That said, generic support for Windows was merged earlier today and the OpenIddict client is fully working on WPF and WinForms. I haven't tested yet on WinUI as there's no .NET generic host support yet, but it's on my radar.

MAUI has a generic-host-like API, but it doesn't support IHostedService so the new OpenIddict Windows integration won't work on MAUI without some modifications.

Don't hesitate to add your voice to dotnet/maui#2244.

@JohnHunterACS
Copy link

@kevinchalet - would you mind if I "borrowed" your ditched code for a proof-of-concept I'm working on? I'm currently accessing an OpenIddict server throw a MAUI WebView, which I realize is deprecated, but none of the WebAuthenticator examples I saw worked properly.

@kevinchalet
Copy link
Member Author

@JohnHunterACS I wouldn't recommend it: this prototype was built for an early version of the OpenIddict client (whose final version was released in December). I'm afraid it won't work without heavy modifications.

@JohnHunterACS
Copy link

😢

@kevinchalet
Copy link
Member Author

@JohnHunterACS see #1657 for an example showing how to make the new generic Windows integration work on MAUI (WinUI-only, of course).

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

Successfully merging this pull request may close these issues.

None yet

3 participants