Skip to content

Commit 7f49efd

Browse files
committed
Merge pull request #109 from bobbyrullo/yes_we_DO_have_TLS
Documentation: remove outdated TLS info
2 parents 5076497 + bfe6cd2 commit 7f49efd

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

Documentation/oidc-notes.md

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,6 @@ Sec. 2. [ID Token](http://openid.net/specs/openid-connect-core-1_0.html#IDToken)
1414
Sec. 3. [Authentication](http://openid.net/specs/openid-connect-core-1_0.html#Authentication)
1515
- Only the authorization code flow (where `response_type` is `code`) is supported.
1616

17-
Sec. 3.1.2. [Authorization Endpoint](http://openid.net/specs/openid-connect-core-1_0.html#AuthorizationEndpoint)
18-
- 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.
19-
2017
Sec. 3.1.2.1. [Authentication Request](http://openid.net/specs/openid-connect-core-1_0.html#AuthRequest)
2118
- max_age not implemented; it's OPTIONAL in the spec, but if it's present servers MUST include auth_time, which dex does not.
2219
- None of the other OPTIONAL parameters are implemented with the exception of:

0 commit comments

Comments
 (0)