-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Documentation: explain how to write an app that uses dex #755
Comments
hello @ericchiang I am new here and would like to look into contributing to docs first, but I need some clarification on this issue:
|
@freefood89 we're going to be rolling a OpenID Connect proxy ourselves. If you want to try express+passport we'd be super happy to include it in our examples. It also helps us test our implementation. |
@ericchiang feel free to assign the issue to me if you can/want to |
@freefood89 I don't think I can (your outside the org). There's a lot to this issue anyway, so I don't think it's a big deal. Please reach out if you need any help or find any issues. I remember one of the frameworks requiring parts of the spec we don't implement. You may want to look at #376 for background. |
Looks like the only working express app I know that uses Dex consumes a hacked passport module called oidc-passport, which I know is not a supported module because it's written by the friend that got it working. I'm contemplating writing a passport module myself, but for now I will look into making an example using flask first. Looking back, it's kind of funny that I was having him write a user API as a companion to Dex. I didn't realize till now that the lack of a userinfo API was causing this much trouble. |
@freefood89 there's a proposal for adding the userinfo endpoint in that issue #376 (comment). If you want to give it a crack, feel free. |
we need an identity provider for our app. we need all the features which were removed in v2. namely
is there a way to implement these in go, so that when you guys are done with adding them back into v2, i can easily just update dex api and these features get handled by dex instead of our custom code? i reckon yall will be adding these features using the gRPC api.. |
@akshatprakash we don't have plans to add those features back in. Dex isn't a user management system, so these are more appropriate for orthogonal apps that want to talk to dex's gRPC API or use it's login flows. E.g. for the registration flows, we would probably add a link to our login screen that send the user to a different app that handles registration. |
@akshatprakash perhaps CoreOS's fork of bit.ly's oauth2 proxy will do what you want? |
@ericchiang also, could you common on your PR on bitly's oauth2_proxy? Seems like they're relatively close to merging it with a small bit of love. Seems worth doing. |
@SEJeff PR updated. Know anyone at bitly to help it get merged? :) |
Brought up by @set321go in #753
This can include:
p1 because this kind of documentation is a blocker for users who want to use dex.
The text was updated successfully, but these errors were encountered: