Skip to content

Commit

Permalink
Merge pull request #109 from bobbyrullo/yes_we_DO_have_TLS
Browse files Browse the repository at this point in the history
Documentation: remove outdated TLS info
  • Loading branch information
bobbyrullo committed Sep 3, 2015
2 parents 5076497 + bfe6cd2 commit 7f49efd
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions Documentation/oidc-notes.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,6 @@ Sec. 2. [ID Token](http://openid.net/specs/openid-connect-core-1_0.html#IDToken)
Sec. 3. [Authentication](http://openid.net/specs/openid-connect-core-1_0.html#Authentication)
- Only the authorization code flow (where `response_type` is `code`) is supported.

Sec. 3.1.2. [Authorization Endpoint](http://openid.net/specs/openid-connect-core-1_0.html#AuthorizationEndpoint)
- In a production system TLS is required but the dex web-server only supports HTTP right now - it is expected that until HTTPS is supported, TLS termination will be handled outside of dex.

Sec. 3.1.2.1. [Authentication Request](http://openid.net/specs/openid-connect-core-1_0.html#AuthRequest)
- max_age not implemented; it's OPTIONAL in the spec, but if it's present servers MUST include auth_time, which dex does not.
- None of the other OPTIONAL parameters are implemented with the exception of:
Expand Down

0 comments on commit 7f49efd

Please sign in to comment.