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

Documentation: OIDC conformance test setup #1050

Merged
merged 2 commits into from Aug 25, 2017

Conversation

estroz
Copy link
Contributor

@estroz estroz commented Aug 25, 2017

Document setting up a dex test server and OpenID test instance to test dex against the OpenID Connect Basic OP (profile outline, section 2.1.1) test profile.

Relates to #42

@rithujohn191
Copy link
Contributor

Once Dex actually passes all the conformance tests we can get this doc in.

@ericchiang
Copy link
Contributor

Once Dex actually passes all the conformance tests we can get this doc in.

I disagree. I think it'd be useful for us to document how to run the conformance tests in order to make dex conform to it.

@rithujohn191
Copy link
Contributor

Ok then this document needs to be re-worded to reflect that. It has to be clear that these are just instructions and we do not fully comply as of yet

@estroz estroz force-pushed the oidc-certification-docs branch 2 times, most recently from 83adc1e to a12f587 Compare August 25, 2017 18:40
@estroz
Copy link
Contributor Author

estroz commented Aug 25, 2017

@rithujohn191 @ericchiang added some sweet tables and noted that dex is not yet compliant

Copy link
Contributor

@rithujohn191 rithujohn191 left a comment

Choose a reason for hiding this comment

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

this is awesome thanks!

@estroz estroz merged commit 9517d17 into dexidp:master Aug 25, 2017
@estroz estroz deleted the oidc-certification-docs branch August 25, 2017 21:40
mmrath pushed a commit to mmrath/dex that referenced this pull request Sep 2, 2019
Documentation: OIDC conformance test setup
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